-
Notifications
You must be signed in to change notification settings - Fork 13
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
Bump up to yui-relayer@v0.5.1 (ibc-go@v8.2.0) #40
Conversation
Signed-off-by: Masanori Yoshida <masanori.yoshida@datachain.jp>
Signed-off-by: Masanori Yoshida <masanori.yoshida@datachain.jp>
@bluele OK. Should I update ibc-solidity to the latest BEFORE this PR? |
I think it should be included in this PR. Because the minimum version of ibc-solidity corresponding ibc-go v8 is v0.3.27. |
go.mod
Outdated
go 1.20 | ||
go 1.21 | ||
|
||
toolchain go1.22.2 |
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.
I am not familiar with go mod, but is this directive required since go 1.21? If yes, is there any reason why that toolchain
minor version differs from the go
directive one?
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.
Signed-off-by: Masanori Yoshida <masanori.yoshida@datachain.jp>
Signed-off-by: Masanori Yoshida <masanori.yoshida@datachain.jp>
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.
@siburu Thanks, LGTM👍
Blocked by #39Bump up dependencies