Skip to content

Commit

Permalink
[crypto] feat: remove BLS12-377 (MystenLabs#5443)
Browse files Browse the repository at this point in the history
* feat: remove BLS12-377

BLS12-377 was introduced in MystenLabs/narwhal#36 as a PoC demonstrating the ability to use Narwhal with https://github.com/celo-org/celo-bls-snark-rs, a specific implementation of BLS12-377.
Due to idiosyncracies of this implementation (outlined in celo-org/celo-bls-snark-rs#228), this required using a fork (https://github.com/huitseeker/celo-bls-snark-rs), which is less than ideal. Further, the org behind the original implementation has other plans (https://github.com/celo-org/bls-crypto).

With MystenLabs/fastcrypto#136, MystenLabs/fastcrypto#143 and MystenLabs/fastcrypto#144, BLS12-377 is supported as part of our fastcrypto library.
The present implementation is obsolete.

* fix: remove obsolete Narwhal-celo crypto tests
  • Loading branch information
huitseeker authored Oct 21, 2022
1 parent 60298af commit 812c841
Show file tree
Hide file tree
Showing 12 changed files with 33 additions and 1,391 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,10 +73,6 @@ jobs:
- name: cargo test
run: |
cargo nextest run --profile ci
- name: narwhal celo crypto tests
run: |
cargo nextest run --features celo --profile ci -p narwhal-crypto
cargo test --doc --features celo -p narwhal-crypto
- name: Doctests
run: |
cargo test --doc
Expand Down
Loading

0 comments on commit 812c841

Please sign in to comment.