-
Notifications
You must be signed in to change notification settings - Fork 42
chore: update flow ideas #61
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
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #61 +/- ##
=======================================
Coverage 95.60% 95.60%
=======================================
Files 42 42
Lines 9754 9754
=======================================
Hits 9325 9325
Misses 429 429 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
noa-starkware
left a comment
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.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @arad-starkware)
src/flow_test/flow_ideas.md line 79 at r1 (raw file):
- staker enter in V0, advance epoch, update balance, advance epoch, update balance, upgrade to V3, attest - staker enter in V1, advance epoch, update balance, advance epoch, update balance, upgrade to V3, attest - staker in V2, update balance staker+update balance pool, upgrade, attest in current epoch, attest in next epoch, attest in next next epoch
lets keep that test, Im not sure we cover the same with the next one.
e8b4e7d to
631614f
Compare
arad-starkware
left a comment
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.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @noa-starkware)
src/flow_test/flow_ideas.md line 79 at r1 (raw file):
Previously, noa-starkware wrote…
lets keep that test, Im not sure we cover the same with the next one.
Done.
noa-starkware
left a comment
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.
Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @arad-starkware)
631614f to
a1ef5f2
Compare
Merge activity
|

This change is
Note
Update flow ideas doc: add a Views section and expand/normalize migration and balance test scenarios.
src/flow_test/flow_ideas.md):Viewswith cases for staker/member balance changes and current-epoch balance views.Written by Cursor Bugbot for commit a1ef5f2. This will update automatically on new commits. Configure here.