Skip to content

chore(light-clients/08-wasm): add local pin back#8904

Merged
srdtrk merged 2 commits intorelease/v11.0.xfrom
serdar/xxx-v11.0.x-post-release
Apr 15, 2026
Merged

chore(light-clients/08-wasm): add local pin back#8904
srdtrk merged 2 commits intorelease/v11.0.xfrom
serdar/xxx-v11.0.x-post-release

Conversation

@srdtrk
Copy link
Copy Markdown
Member

@srdtrk srdtrk commented Apr 14, 2026

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
Copy Markdown

greptile-apps bot commented Apr 14, 2026

Greptile Summary

This chore PR re-adds the local replace directive (github.com/cosmos/ibc-go/v11 => ../../../) to modules/light-clients/08-wasm/go.mod, so the 08-wasm sub-module tracks the local root module during ongoing development on the release/v11.0.x branch, and updates go.sum accordingly. A minor CHANGELOG.md formatting fix is included in the second commit.

Confidence Score: 5/5

  • This PR is safe to merge — it is a routine dependency management change with no logic modifications.
  • All three files contain low-risk changes: re-adding a standard local replace directive in go.mod, the corresponding go.sum update (which correctly omits checksum entries for the replaced module), and a minor CHANGELOG typo fix. The only finding is a P2 formatting nit in CHANGELOG.md that does not affect correctness or build behaviour.
  • No files require special attention.

Important Files Changed

Filename Overview
modules/light-clients/08-wasm/go.mod Adds github.com/cosmos/ibc-go/v11 => ../../../ back to the replace block so the sub-module uses the local root module; standard practice for multi-module repos on a release branch.
modules/light-clients/08-wasm/go.sum go.sum updated to reflect the local replace; correctly contains no checksum entries for github.com/cosmos/ibc-go/v11 (expected when a local path replace is active).
CHANGELOG.md Minor typo fix; one pre-existing formatting inconsistency on line 64 (missing space between component tag and issue link) remains but is outside the scope of this PR.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["modules/light-clients/08-wasm\n(separate Go module)"]
    B{"go.mod replace directive"}
    C["github.com/cosmos/ibc-go/v11 v11.0.0\n(published release)"]
    D["../../../\n(local path — this repo)"]

    A --> B
    B -->|"Before PR\n(no replace)"| C
    B -->|"After PR\n(local pin added back)"| D

    style D fill:#90EE90
    style C fill:#FFD700
Loading

Reviews (1): Last reviewed commit: "docs: fix typo" | Re-trigger Greptile

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (release/v11.0.x@70b6864). Learn more about missing BASE report.

Additional details and impacted files
@@                Coverage Diff                 @@
##             release/v11.0.x    #8904   +/-   ##
==================================================
  Coverage                   ?   67.23%           
==================================================
  Files                      ?      288           
  Lines                      ?    15417           
  Branches                   ?        0           
==================================================
  Hits                       ?    10365           
  Misses                     ?     4432           
  Partials                   ?      620           
Flag Coverage Δ
08-wasm 65.04% <ø> (?)
ibc-go 67.31% <ø> (?)

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.

Comment thread CHANGELOG.md
* [\#8864](https://github.com/cosmos/ibc-go/pull/8864) chore(deps): bump sdk dependencies to latest rcs (backport of [\#8863](https://github.com/cosmos/ibc-go/pull/8864))
* [\#8864](https://github.com/cosmos/ibc-go/pull/8864) chore(deps): bump sdk dependencies to latest rcs (backport of [\#8863](https://github.com/cosmos/ibc-go/pull/8863))
* [\#8820](https://github.com/cosmos/ibc-go/pull/8820) imp!: bump ibc-go to v11 and use sdk.ValidateAuthority
* (light-clients/08-wasm)[\#8500](https://github.com/cosmos/ibc-go/pull/8500) Bump **github.com/prysmaticlabs/prysm/v5** to **github.com/OffchainLabs/prysm/v6@v6.0.4**
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Missing space in component tag

This entry is missing a space between the component tag and the issue link, which is inconsistent with every other entry in the section. Since this PR already includes a CHANGELOG typo fix, it's a good opportunity to address this too.

Suggested change
* (light-clients/08-wasm)[\#8500](https://github.com/cosmos/ibc-go/pull/8500) Bump **github.com/prysmaticlabs/prysm/v5** to **github.com/OffchainLabs/prysm/v6@v6.0.4**
* (light-clients/08-wasm) [\#8500](https://github.com/cosmos/ibc-go/pull/8500) Bump **github.com/prysmaticlabs/prysm/v5** to **github.com/OffchainLabs/prysm/v6@v6.0.4**

@srdtrk srdtrk merged commit 4a8c12a into release/v11.0.x Apr 15, 2026
92 of 93 checks passed
@srdtrk srdtrk deleted the serdar/xxx-v11.0.x-post-release branch April 15, 2026 11:52
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