-
Notifications
You must be signed in to change notification settings - Fork 198
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
Feat/round activation #3619
Feat/round activation #3619
Conversation
# Conflicts: # process/errors.go
…oundManager_notification # Conflicts: # process/roundActivation/roundActivation.go # process/roundActivation/roundActivation_test.go
…nd_activation Round Activation handler
…ification En 11019 round manager notification
Codecov Report
@@ Coverage Diff @@
## development #3619 +/- ##
===============================================
+ Coverage 73.67% 73.69% +0.01%
===============================================
Files 585 587 +2
Lines 75435 75584 +149
===============================================
+ Hits 55578 55698 +120
- Misses 15448 15471 +23
- Partials 4409 4415 +6
Continue to review full report at Codecov.
|
# Conflicts: # process/errors.go
…vation Merge dev into round activation
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.
…ound-activation Merge new dev into feat round activation
Added
RoundActivationHandler
which:IsEnabledInRound(name string, round uint64)
or withIsEnabled
-> which checks if the flag is enabled in the currently processed roundRoundConfirmed
and updates its own internal "currently processing round"enableRound.toml
which contains all round activations (for now -> dummy)RoundNotifier
andRoundSubscriberHandler
interfaces, which resembleEpochNotifier
andEpochSubscriberHandler
roundNotifier
implementation ofRoundNotifier
RoundNotifier
incoreComponents.go
roundActivationHandler
toroundNotifier
inbootstrapComponents.go