-
Notifications
You must be signed in to change notification settings - Fork 176
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
Epoch setup random source update #5088
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #5088 +/- ##
==========================================
- Coverage 55.99% 55.65% -0.35%
==========================================
Files 649 994 +345
Lines 63379 95050 +31671
==========================================
+ Hits 35492 52900 +17408
- Misses 25321 38177 +12856
- Partials 2566 3973 +1407
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This PR will be open once onflow/flow-core-contracts#397 is merged |
@tarakby The core contracts PR is merged and is getting upgraded on mainnet tomorrow. Does anything need to be done about this PR? |
@joshuahannan the only constraint is to not deploy this PR on Mainnet till the contract is upgraded. This is guaranteed once the contract is upgraded so I'll open this PR tomorrow. |
Co-authored-by: Jordan Schalm <jordan@dapperlabs.com>
This PR is part of improving the security of the epoch source of randomness after updating the epoch core-contract (onflow/flow-core-contracts#397).