ref-fvm
ref-fvm copied to clipboard
Add new accepted hash functions to the `hash` syscall (sha256, keccak256, ripemd160)
Codecov Report
Merging #703 (531cb65) into master (f144970) will decrease coverage by
0.07%. The diff coverage is0.00%.
@@ Coverage Diff @@
## master #703 +/- ##
==========================================
- Coverage 50.92% 50.85% -0.08%
==========================================
Files 120 121 +1
Lines 9691 9705 +14
==========================================
Hits 4935 4935
- Misses 4756 4770 +14
| Impacted Files | Coverage Δ | |
|---|---|---|
| fvm/src/kernel/default.rs | 15.97% <0.00%> (+0.15%) |
:arrow_up: |
| fvm/src/syscalls/crypto.rs | 0.00% <0.00%> (ø) |
|
| sdk/src/crypto.rs | 0.00% <0.00%> (ø) |
|
| shared/src/crypto/hash.rs | 0.00% <0.00%> (ø) |
Can we get a new integration test?
@mriise could you fix the clippy issue and the conflicts?
note: it is very odd that clippy was silent about this locally, maybe it was removed in latest toolchain update?
For context ripemd160 was added as it is used by Bitcoin for address generation (for some random reason).