-
-
Notifications
You must be signed in to change notification settings - Fork 306
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
chore: release v1.17.0 #6523
Merged
Merged
chore: release v1.17.0 #6523
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
* refactor: update retry function * chore: subtract 1 from retry values * Keep same number of retries in download test * Update retry function * Fix jsonRpcHttpClient tests * Restore comment --------- Co-authored-by: Nico Flaig <nflaig@protonmail.com>
* refactor: reuse command types/utils across packages * Allow as an alternative to setting a default * Update prover cmd options * Update comment * Fix comment * Remove unused import * Demand option if arg must be provided * Update handler
ci: fix downloading chrome in browser tests
* Add block rewards api * Add test * Add unit test * Lint * Address comment * Reduce code redundancy * Read reward cache first before calculate * Lint * Partially address comments * Accept optional postState to get the reward cache * Update test * lint * Update packages/beacon-node/src/chain/rewards/blockRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Update packages/beacon-node/src/chain/rewards/blockRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Update packages/beacon-node/src/chain/rewards/blockRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Update packages/beacon-node/src/chain/rewards/blockRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Rename proposerRewards to blockRewards. Fix import * Remove getBlockRewards from api ignore list * Fix test * Rename state to preState * Add description to fields in BlockRewards * Clean up imports * Use jsdoc to document properties * Apply suggestions from code review * Add `getPreStateSync()` * fix: clone states to compute block rewards --------- Co-authored-by: Nico Flaig <nflaig@protonmail.com> Co-authored-by: Cayman <caymannava@gmail.com> Co-authored-by: Tuyen Nguyen <vutuyen2636@gmail.com>
* chore: clean up prettier-ignore in tests * setConfig needs to be called in describe * Set test timeout as hookTimeout * Rename variable * Remove unrelated change * Better name for min capella time variable
* chore: add eslint rule to prevent unused expressions * Fix unit tests
* Request retries are abortable * Reduce timeouts in tests * Abort retry if not first attempt
* fix: increase bodyLimit of the beacon api server * update limit
#6478) remove extra validation from builder response in builder proposal
* Add new assertions for multi-fork sim tests * Remove redundant sim-merge test * Remove the unused script file and update docs * Fix an edge case for multi-client tests * Remove unused ci tasks * Enable all nodes
* chore: cleanup older dependencies * chore: some more removed dependencies
* fix: update events test data * fix: reflect typo fix * fix: remove workarounds * chore: remove now irrelevan dropOneOf option * Fix parent block number in SSE payload attributes * Tests pass against latest spec version * Update comment * Lint * Update beacon api spec to v2.5.0 --------- Co-authored-by: Nico Flaig <nflaig@protonmail.com>
* Update typescript * Fix lint errors
* chore: detect when docker is not installed * chore: improve e2e tests requirement
* Upgrade eslint and prettier * Fix the lint errors
* chore: add eslint rule to restrict global fetch * Add comment to eslint disable * Rephrase comment
* Delete lodash.pick package * Lint --------- Co-authored-by: Nico Flaig <nflaig@protonmail.com>
* update vc-configuration md doc * Grammar correction Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Show only one ref on text Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Put create keystore under setup validator --------- Co-authored-by: Nico Flaig <nflaig@protonmail.com>
* feat: regen to consume state cache reload api * chore: address PR comments
* Add block rewards api * Add test * Add unit test * Lint * Address comment * Reduce code redundancy * Read reward cache first before calculate * Lint * Add endpoint definition for sync rewards * Add calculation logic * Lint * Follow convention from block rewards * Include getSyncCommitteeRewards in unit test * Update packages/beacon-node/src/chain/rewards/syncCommitteeRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Update packages/beacon-node/src/api/impl/beacon/rewards/index.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Improve filtering logic * Early throw on empty preState in getBlockRewards * Add jsdoc * Address comment * Clarify comment * Address comment * Update packages/beacon-node/src/chain/rewards/syncCommitteeRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * Improve naming of filters * Lint * Update packages/beacon-node/src/chain/rewards/syncCommitteeRewards.ts Co-authored-by: Nico Flaig <nflaig@protonmail.com> * ids -> validatorIds --------- Co-authored-by: Nico Flaig <nflaig@protonmail.com>
* feat: aggressively pull blobs as soon as we see the block * fixes * some logging improvement * improve log * fix tsc * improve comments * apply feedback * refactor to a separate event and handler for unknownblock input * fix e2e * reduce diff * add metric to track availability resolution * fix circular include issue
…#6504) * feat: new getCheckpointStateOrBytes() api and clone states from cache * fix: clone states without transferring cache * fix: persistentCheckpointsCache.test.ts unit test * fix: remove .only in persistentCheckpointsCache.test.ts
* feat: clone option for state caches * fix: compilation error
* chore: simplify LightClient usage * chore: lint * chore: lint
* feat: use Uint32Array for shuffling/committees * chore: address pr comments
Performance Report✔️ no performance regression detected Full benchmark results
|
nflaig
approved these changes
Mar 11, 2024
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## stable #6523 +/- ##
==========================================
- Coverage 61.65% 61.53% -0.12%
==========================================
Files 553 556 +3
Lines 57998 58647 +649
Branches 1843 1847 +4
==========================================
+ Hits 35756 36087 +331
- Misses 22206 22520 +314
- Partials 36 40 +4 |
🎉 This PR is included in v1.17.0 🎉 |
Closed
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.
Motivation
Re-releasing v1.17.0 with RC.2 at commit cae26be. This supersedes PR #6508 .