-
Notifications
You must be signed in to change notification settings - Fork 166
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
[saya] Publish proof to Celestia #2053
[saya] Publish proof to Celestia #2053
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2053 +/- ##
==========================================
+ Coverage 68.21% 68.49% +0.27%
==========================================
Files 323 328 +5
Lines 40269 41058 +789
==========================================
+ Hits 27468 28121 +653
- Misses 12801 12937 +136 ☔ View full report in Codecov by Sentry. |
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.
Great work @neotheprogramist! Minor comments to be addressed.
state_diff_proof.iter().flat_map(|fe| fe.to_bytes_be().to_vec()).collect(); | ||
let proof_blob = Blob::new(self.namespace, proof_bytes)?; | ||
|
||
// TODO: we may want to use `blob_get` to ensure the state diff has been published |
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.
If you're ok for doing it here, let's do it. If not, let's open an issue to keep track of that in the future.
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.
Thanks for the work here @neotheprogramist!
Description
Including a proof blob into celestia DA.
Tests
As a local celestia instance is needed, tested as e2e.
Added to documentation?
Already present, only internal implementation changed.
Checklist
scripts/prettier.sh
,scripts/rust_fmt.sh
,scripts/cairo_fmt.sh
)scripts/clippy.sh
,scripts/docs.sh
)