-
Notifications
You must be signed in to change notification settings - Fork 26
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): merge iliad client changes to odyssey #199
Merged
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
leeren
requested review from
LeoHChen,
AndyBoWu,
jdubpark,
edisonz0718,
ezreal1997 and
limengformal
as code owners
October 15, 2024 19:00
leeren
force-pushed
the
chore/merge-iliad-changes
branch
from
October 15, 2024 19:33
07b58f8
to
7729713
Compare
leeren
changed the title
merge iliad changes to odyssey
chore(release): merge iliad client changes to odyssey
Oct 15, 2024
leeren
force-pushed
the
chore/merge-iliad-changes
branch
from
October 15, 2024 19:54
ca3ed73
to
88e4e43
Compare
add cometBFT's one block rollback cmd
* fix(x/evmstaking): branch condition on max spendable * test(x/evmstaking): add expected call in tests
* feat(cli): add unjail validator subcommand * fix: use static predeploy ca in validator logic
increased test coverage to 100% changes validate functions use concrete type instead of any. there is no reasoning for using any. rename withdraw test suite to avoid name conflict with param test suite
increased test coverage to 100%
increased coverage to 71.4%
it should append QueueElementsPrefixSuffix when query elements of queue it fixes #84
* add test cases to db
* add test cases for helper increased test coverage to 100% * fix ci
increased test coverage to 89.7% add ci rule to not use t.Parallel to avoid data race issue for cosmos orm table.
increase coverage to 83.3% moved common function to keeper_internal_test.go
increased coverage to 77.3%
increased coverage to 100%
* add version txt * fixed task name * fixed url * fixed url * opti title * finish test * fixed platform * fixed pre-commit
cmtAPI is lazily set, so during replyBlocks it is nil.
changes of mockEngineAPI - forceInvalidNewPayloadV3 and forceInvalidForkchoiceUpdatedV3 are added to simulate failed apis changes of engineMock - Add storeKey to make engineMock's methods dependent on sdk.Context for better testability NewBlock - Because of above changes, we need to rlp encode and decode block data. But if we create a block with nil withdrawals and nil withdrawalHash, rlp doesn't work well (couldn't figure out the root cause) - To avoid rlp error, pass non-nil withdrawals so withdrawalHash can be set as non-nil emptyHash value.
increased coverage to 83.8% added mock getPayloadV3Func for mocking getPayloadV3
* feat(mint): port mint module from cosmos sdk v0.50.7 * feat(mint): port mint module from cosmos sdk v0.50.7 * feat(mint): port mint module from cosmos sdk v0.50.7 * feat(mint): port mint module from cosmos sdk v0.50.7 * feat(mint): remove useless parameters * feat(mint): new inflation function * feat(mint): mint parameters change event log processing * feat(mint): mint module api * feat(mint): placeholder for param change events * feat(mint): fix unit test * feat(mint): fix based on comments * feat(mint): fix based on comments * feat(mint): update readme * feat(mint): update readme
* Change UUPS by TransparentUpgradeableProxy, use etching to generate initial contracts * etch all the contracts * git ignore local dumps * fix contracts/script/EtchInitialState.s.sol, use it also as setup for tests * fix GenerateAlloc for UpgradeEntryPoint, add allocations and test upgradeability * lint and cl fixes * temporarily disabled solhint in CI/CD * temporarily remove run_solidity_lint * temporarily remove lint form workflow * remove need for env function in test
Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.8.0 to 1.8.1. - [Release notes](https://github.com/spf13/cobra/releases) - [Commits](spf13/cobra@v1.8.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jongwon Park <jwparktom@gmail.com>
* feat(api): add epochs api * feat(api): add epoch info api
* feat(mint): update param type * feat(mint): make param value consistent among example and tests
* feat(cli): add key conversion subcommand * adds uncompressed pubkey (hex) as input
* fix(solhint): rm redundant pre-commit linting hook * fix(solhint): update linting script & pkg version * chore(contracts): apply linting fixes
leeren
force-pushed
the
chore/merge-iliad-changes
branch
from
October 15, 2024 20:25
88e4e43
to
746a040
Compare
limengformal
approved these changes
Oct 15, 2024
ezreal1997
approved these changes
Oct 15, 2024
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.
LGTM
Binary uploaded successfully 🎉📦 Version Name: 0.12.0-unstable-f51e781 |
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.
merges all non-hardfork related client changes from iliad to odyssey main branch
issue: #200