Skip to content
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

#4045 re-enable epochs tests with dynamic quora #4103

Merged
merged 2 commits into from
Feb 6, 2025
Merged

#4045 re-enable epochs tests with dynamic quora #4103

merged 2 commits into from
Feb 6, 2025

Conversation

pls148
Copy link
Contributor

@pls148 pls148 commented Feb 3, 2025

Closes #4045, closes #4046, closes #4047, closes #4048, closes #4049, closes #4050

This PR:

This PR does not:

Key places to review:

@pls148 pls148 requested a review from bfish713 as a code owner February 3, 2025 20:13
@pls148 pls148 marked this pull request as draft February 3, 2025 20:13
@pls148 pls148 force-pushed the ps/4045 branch 10 times, most recently from db283db to 847fd7f Compare February 4, 2025 22:54
* #4045 re-enable epochs tests with dynamic quora

* Remove Epoch 0

* Allow dead code for genesis method for EpochNumber

The method is not called at the moment but it's better
to have it because otherwise the default genesis method
returning 0 might be called by accident.

* Adjust test_genesis_epoch_from_version

---------

Co-authored-by: pls148 <[email protected]>
@pls148 pls148 marked this pull request as ready for review February 6, 2025 17:27
@pls148 pls148 merged commit e4abb5c into main Feb 6, 2025
25 checks passed
@pls148 pls148 deleted the ps/4045 branch February 6, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants