-
Notifications
You must be signed in to change notification settings - Fork 26
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
chore(starknet_api): use const fn create for testing block info #2561
chore(starknet_api): use const fn create for testing block info #2561
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## arni/test_utils/move_to_snapi #2561 +/- ##
================================================================
Coverage ? 47.40%
================================================================
Files ? 29
Lines ? 2909
Branches ? 2909
================================================================
Hits ? 1379
Misses ? 1464
Partials ? 66 ☔ 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.
Reviewed all commit messages.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @ArniStarkware)
crates/starknet_api/src/test_utils.rs
line 180 at r1 (raw file):
impl BlockInfo { pub const fn create_for_testing() -> Self {
Why do we need this to be const?
763ff23
to
120ccfc
Compare
27c2d54
to
f480b25
Compare
120ccfc
to
0b9b241
Compare
f480b25
to
1ba7076
Compare
Benchmark movements: |
0b9b241
to
534fc0c
Compare
1ba7076
to
2f66a0d
Compare
534fc0c
to
9896be8
Compare
2f66a0d
to
bcbd6b9
Compare
No need for this PR. |
No description provided.