-
Notifications
You must be signed in to change notification settings - Fork 551
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
feat: update ts-relayer version #3601
Conversation
This PR was intended to update confio/ts-relater to the latest version to fix issue #3541. The reported issue won't be fixed with this PR because |
This new version contains a fix to make it work with the lates Tendermint version.
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.
Looks good.
However it seems only part of the deps have been updated in the lock file (shouldn't matter much as cosmjs is also explicitly versioned in our package.json) + the lock file is version 3 (i assumed you npm installed with node 18)
Since we target node 16 in the pkg nodetime, I think it's best if you:
- completely remove node_modules and package-lock.json
- run npm install using node v16 (I suggest using https://github.com/nvm-sh/nvm)
- run './scripts/gen-nodetime` and build CLI again
- test
- commit the resulting package-lock if all fine
It makes sense, thanks @clockworkgr 👍 |
This PR fixes
ignite relayer
command:ts-relayer
tov0.10.0
(adds support for CometBFT)cosmjs
tov0.31.1
(fixes auto gas multiplier issue)