-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor accusations #31
Refactor accusations #31
Conversation
bridge/test/accusations/invalid-transaction-consumption-accusations.test.ts
Outdated
Show resolved
Hide resolved
2c01be6
to
e0d5cd4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can merge this to main as soon as pipeline is green
bridge/test/accusations/invalid-transaction-consumption-accusations.test.ts
Outdated
Show resolved
Hide resolved
Shortened contract name to fix salt generation
Generated Immutable auth
Updated accusation error codes
e0d5cd4
to
2934b26
Compare
Codecov Report
@@ Coverage Diff @@
## main #31 +/- ##
==========================================
- Coverage 45.62% 45.61% -0.02%
==========================================
Files 214 214
Lines 38503 38503
==========================================
- Hits 17568 17563 -5
- Misses 19005 19006 +1
- Partials 1930 1934 +4
Continue to review full report at Codecov.
|
Scope
salt-type
parameter for immutable factory, allowing for a defined contract type to be included in the salt calculationWhy?
MP-590
MP-551
Todos
If any, what are the follow-up tasks required other than merging this PR? Have they been arranged?