-
Notifications
You must be signed in to change notification settings - Fork 857
Migrate sei-wasmd into sei-chain as mono repo
#2400
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
Conversation
Add mutex around WasmVM write calls
FORKED_CHANGELOG
Use HexBytes without custom Marshal/UnmarshalJSON functions
execute contract metrics
Replace cosmos/tendermint/iavl/tmdb/ibc dependencies with forked versions
Bump wasmvm version for cherry
refactor tx position and update usages
Add timeout to create
* Smart Contract Query Metrics * Add more keys
* Remove Non-Multiplier Gas Meter * Remove go mod changes * Remove go sum changes
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.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
590ea18 to
68be20b
Compare
Limit additional changes by moving wasmd tests as they are to CI workflow.
Sei-chain has protos with no CI integrated builds... Separating this debt work into another PR to limit changes here.
Because, sei-wasmd is no longer an externally fetched dependency.
6709d60 to
9045827
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because the head coverage (51.89%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #2400 +/- ##
==========================================
- Coverage 61.26% 51.89% -9.37%
==========================================
Files 382 464 +82
Lines 35316 45388 +10072
==========================================
+ Hits 21635 23556 +1921
- Misses 12146 20015 +7869
- Partials 1535 1817 +282
🚀 New features to boost your workflow:
|
Test prior to the work linked below were broken in seiwasmd: * sei-protocol/sei-wasmd#80 Postponing the wasmd CI test runs until that work is independently migrated over.
Merge git commit history from
sei-wasmdintosei-chainand update the replace directive in root ofsei-chainto use relative path.Note
The merged commits from
sei-wasmdpoint tov0.3.9, consistent with the version bubbled up tosei-chainprior to this migration. Note that 4 commits fromsei-wasmdare excluded, where 2 of which carry functional significance. Due to issues detected during the bubble up related to high gas usage the work here explicitly excluded those commits. A seprate set of PRs will re-introduce the excluded commits to isolate risk from mono-repo migration.The PRs that need to be seprately bubbled up to
sei-chainare: