-
Notifications
You must be signed in to change notification settings - Fork 951
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
Generate Tx Borsh schema #2931
Generate Tx Borsh schema #2931
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2931 +/- ##
==========================================
- Coverage 53.44% 53.44% -0.01%
==========================================
Files 310 310
Lines 101574 101580 +6
==========================================
+ Hits 54288 54289 +1
- Misses 47286 47291 +5 ☔ View full report in Codecov by Sentry. |
09dc566
to
582537d
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.
a couple of suggestions
582537d
to
f173228
Compare
f173228
to
3636c41
Compare
* murisi/generate-tx-schema: Added a changelog entry. Generate Tx Borsh schema using the latest MASP schema, whilst avoiding name collissions.
* origin/murisi/generate-tx-schema: Added a changelog entry. Generate Tx Borsh schema using the latest MASP schema, whilst avoiding name collissions. # Conflicts: # crates/apps/src/lib/bench_utils.rs # crates/apps/src/lib/node/ledger/shell/finalize_block.rs # crates/apps/src/lib/node/ledger/shell/prepare_proposal.rs # crates/apps/src/lib/node/ledger/shell/process_proposal.rs # crates/sdk/src/tx.rs # crates/tx/src/data/mod.rs
Describe your changes
Made the necessary changes to successfully generate an accurate
Tx
Borsh schema. This is useful for describing the transaction format to external collaborators. More specifically, the changes were as follows:Tx
schema to a fileNote: Because the MASP crate update fixes serialization of a certain
Transaction
component in corner cases, this PR is breaking.Indicate on which release or other PRs this topic is based on
Namada v0.32.0
Checklist before merging to
draft