-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add logic to check for sync committee duties #4
Merged
TobiWo
merged 17 commits into
main
from
feature/add-logic-to-check-for-sync-committee-duties
Dec 23, 2022
Merged
Add logic to check for sync committee duties #4
TobiWo
merged 17 commits into
main
from
feature/add-logic-to-check-for-sync-committee-duties
Dec 23, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TobiWo
force-pushed
the
feature/add-logic-to-check-for-sync-committee-duties
branch
from
December 4, 2022 12:41
6fb6b7d
to
f286898
Compare
TobiWo
force-pushed
the
feature/add-logic-to-check-for-sync-committee-duties
branch
2 times, most recently
from
December 17, 2022 09:30
75246c8
to
8cab26c
Compare
LuisNaldo7
reviewed
Dec 22, 2022
…ve default duty values
* Add sync committee duties to logic * Add function to check whether current duties are outdated
* Add function to fetch and parse sync committee duties * Replace ethereum constants with protocol package * Move ethereum related functions to protocol package * Update functions for extended data types
* Move constants module to to root package * Add graceful killer helper as constant
* Rename module * Get genesis time from protocol constant
* Delete unnecessary class structure * Import provided user arguments directly * Move function to send requests to beacon client to separate module in protocol package * Move fetching genesis time to protocol module * Use genesis time from protocol constants * Delete unnecessary pylint disable * Rename epoch variable * Update docstrings
* Call updated fetcher functions * Delete unnecessary functions * Update imports and use GracefulKiller constant * Update docstrings * Delete unnecessary pylint disable
TobiWo
force-pushed
the
feature/add-logic-to-check-for-sync-committee-duties
branch
from
December 23, 2022 09:53
339f713
to
08f5d41
Compare
* Add sync committee picture * Delete missing sync committee duty notes * Delete spaces * Update todo list
TobiWo
force-pushed
the
feature/add-logic-to-check-for-sync-committee-duties
branch
from
December 23, 2022 10:01
08f5d41
to
94e6f8f
Compare
LuisNaldo7
approved these changes
Dec 23, 2022
TobiWo
deleted the
feature/add-logic-to-check-for-sync-committee-duties
branch
December 24, 2022 09:09
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds all logic to check for current or upcoming sync committee duties which might impact a scheduled maintenance
What was added
What changed