Skip to content

chore(deps): bump ibc-go/v11 to rc.0 tag in 08-wasm#8865

Open
mattac21 wants to merge 2 commits intomainfrom
ma/08-wasm-bump
Open

chore(deps): bump ibc-go/v11 to rc.0 tag in 08-wasm#8865
mattac21 wants to merge 2 commits intomainfrom
ma/08-wasm-bump

Conversation

@mattac21
Copy link
Collaborator

Description

closes: #XXXX


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
  • Include changelog entry when appropriate (e.g. chores should be omitted from changelog).
  • Wrote unit and integration tests if relevant.
  • Updated documentation (docs/) if anything is changed.
  • Added godoc comments if relevant.
  • Self-reviewed Files changed in the GitHub PR explorer.
  • Provide a conventional commit message to follow the repository standards.

@greptile-apps
Copy link

greptile-apps bot commented Mar 25, 2026

Greptile Summary

This PR bumps the github.com/cosmos/ibc-go/v11 dependency in the 08-wasm light client module from a pseudo-version commit (v11.0.0-20260325172943-4f9ef9d030d6) to the official v11.0.0-rc.0 release tag, and removes the local workspace replace directive that was pointing to ../../../. The go.sum is updated accordingly with the correct checksums for the published release.

  • Removes the replace github.com/cosmos/ibc-go/v11 => ../../../ directive, decoupling 08-wasm from the local workspace and pinning it to the published RC tag.
  • The goleveldb replace directive is preserved as expected.
  • go.sum gains the two new checksum entries for v11.0.0-rc.0; no prior entries need removal since the old pseudo-version was under a local replace (which carries no go.sum entries).
  • Note: with the local replace removed, local changes to the root ibc-go/v11 module will no longer be automatically picked up by 08-wasm builds — developers working across both modules will need to temporarily re-add the replace directive. This appears to be intentional for the RC release.

Confidence Score: 5/5

  • This PR is safe to merge — it is a clean, minimal dependency bump with correctly updated checksums and no functional code changes.
  • The change is straightforward: remove the local replace directive and pin to the published v11.0.0-rc.0 tag. The go.sum update is correct (no prior pseudo-version entries to remove, new RC entries properly added). No logic, tests, or APIs are changed. The pattern is consistent with how release candidates are typically prepared in Go monorepos.
  • No files require special attention.

Important Files Changed

Filename Overview
modules/light-clients/08-wasm/go.mod Removes the local replace directive for github.com/cosmos/ibc-go/v11 => ../../../ and bumps the dependency from a pseudo-version commit (v11.0.0-20260325172943-4f9ef9d030d6) to the official v11.0.0-rc.0 release tag. The goleveldb replace directive is retained correctly.
modules/light-clients/08-wasm/go.sum Adds the checksum entries for github.com/cosmos/ibc-go/v11 v11.0.0-rc.0. No removal of the old pseudo-version is needed since that version was used with a local replace directive and therefore had no go.sum entries.

Reviews (1): Last reviewed commit: "use rc.0 tag in 08-wasm" | Re-trigger Greptile

@codecov
Copy link

codecov bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.57%. Comparing base (c0ba765) to head (2572f75).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8865   +/-   ##
=======================================
  Coverage   66.57%   66.57%           
=======================================
  Files         326      326           
  Lines       17157    17157           
=======================================
  Hits        11423    11423           
  Misses       5036     5036           
  Partials      698      698           
Flag Coverage Δ
08-wasm 65.04% <ø> (ø)
ibc-go 66.63% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

github.com/cosmos/gogoproto v1.7.2
github.com/cosmos/ibc-go/modules/light-clients/08-wasm/v11 v11.0.0-20260304204640-9f835275a381
github.com/cosmos/ibc-go/v11 v11.0.0-20260325172943-4f9ef9d030d6
github.com/cosmos/ibc-go/v11 v11.0.0-rc.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this go.mod needed to be tidied in order to be built correctly which bumped the version, but we are still using the local replace here for ibc-go/v11

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.

1 participant