Skip to content
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

Final3 #4

Open
wants to merge 44 commits into
base: main
Choose a base branch
from
Open

Final3 #4

wants to merge 44 commits into from

Conversation

leeren
Copy link
Owner

@leeren leeren commented Oct 10, 2024

issue:

jdubpark and others added 30 commits October 10, 2024 01:02
add cometBFT's one block rollback cmd
* fix(x/evmstaking): branch condition on max spendable

* test(x/evmstaking): add expected call in tests
* deploy script for iptokenslashing

* use solady CREATE3 to deploy IPTokenSlashing deterministically

* fix create3 deployment

* use create2 to deploy create3 deployer

* use predeploy create3, make sure admin is not deployer
* 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 piplabs#84
* 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
zsystm and others added 14 commits October 10, 2024 01:03
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] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jongwon Park <[email protected]>
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants