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

Bump up ibc-go to v8.2.0 #137

Merged
merged 4 commits into from
Apr 10, 2024
Merged

Bump up ibc-go to v8.2.0 #137

merged 4 commits into from
Apr 10, 2024

Conversation

siburu
Copy link
Contributor

@siburu siburu commented Apr 8, 2024

  • Upgrade ibc-go to v8.2.0
  • Fix simapp based on ibc-go's simapp
  • Fix CI (github actions) settings
  • Fix docker compose settings

@siburu siburu force-pushed the ibc-go-v8_2 branch 4 times, most recently from 6e79988 to 7408e54 Compare April 10, 2024 01:30
siburu added 4 commits April 10, 2024 14:59
Signed-off-by: Masanori Yoshida <[email protected]>
- avoid adding the mock client to `AllowedClients` if "*" is already specified
- minimize diff between yui-relayer's simapp with ibc-go's simapp

Signed-off-by: Masanori Yoshida <[email protected]>
Signed-off-by: Masanori Yoshida <[email protected]>
- bump up the versions of the github actions
- execute checkout before setup-go for cache mechanism to work properly

Signed-off-by: Masanori Yoshida <[email protected]>
@siburu siburu marked this pull request as ready for review April 10, 2024 06:02
@siburu siburu requested a review from a team as a code owner April 10, 2024 06:02
@siburu siburu requested a review from bluele April 10, 2024 06:02
@bluele
Copy link
Member

bluele commented Apr 10, 2024

I understand this is out of scope, but as of 2024, the following docker-compose should be replaced with docker compose.

DOCKER_COMPOSE ?= docker-compose

Copy link
Member

@bluele bluele left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM👍

@siburu siburu merged commit f854498 into hyperledger-labs:main Apr 10, 2024
6 checks passed
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.

2 participants