Conversation
carte7000
requested review from
0xAustinWang,
KodeyThomas,
agusaldasoro,
chris-de-leon-cll,
Copilot,
mateusz-sekara,
tt-cll and
winder
There was a problem hiding this comment.
Pull request overview
Pins the github.com/smartcontractkit/chainlink-ccip dependency set to a newer develop commit across the repo’s Go modules, updating corresponding go.sum entries and removing a now-unneeded local replace in the CCV EVM deployment submodule.
Changes:
- Bump
chainlink-ccip,chainlink-ccip/deployment, andchainlink-ccip/ccv/chains/evm(where applicable) to20260306193229-66635470b83d. - Refresh
go.sumchecksums to match the updated pseudo-versions. - Remove the local
replace github.com/smartcontractkit/chainlink-ccip/deployment => ../../../../deploymentfromccv/chains/evm/deployment/go.mod.
Reviewed changes
Copilot reviewed 4 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| integration-tests/go.mod | Bumps pinned CCIP versions used by the integration-tests module. |
| integration-tests/go.sum | Updates checksum entries for the bumped ccv/chains/evm module. |
| devenv/go.mod | Bumps CCIP and deployment versions for the dev environment module. |
| devenv/go.sum | Updates checksum entries for the bumped chainlink-ccip module. |
| chains/evm/deployment/go.mod | Bumps CCIP and deployment versions for the EVM deployment module. |
| chains/evm/deployment/go.sum | Updates checksum entries for the bumped chainlink-ccip module. |
| ccv/chains/evm/deployment/go.mod | Removes local replace and bumps CCIP-related requirements to the new commit. |
| ccv/chains/evm/deployment/go.sum | Updates checksum entries for bumped CCIP submodules (including deployment). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| github.com/smartcontractkit/chainlink-ccip v0.1.1-solana.0.20260306193229-66635470b83d | ||
| github.com/smartcontractkit/chainlink-ccip/chains/evm/deployment v0.0.0-00010101000000-000000000000 | ||
| github.com/smartcontractkit/chainlink-ccip/chains/solana v0.0.0-20260121163256-85accaf3d28d | ||
| github.com/smartcontractkit/chainlink-ccip/chains/solana/deployment v0.0.0-00010101000000-000000000000 | ||
| github.com/smartcontractkit/chainlink-ccip/chains/solana/gobindings v0.0.0-20251014191100-bad58388f0c9 | ||
| github.com/smartcontractkit/chainlink-ccip/deployment v0.0.0-20260304181639-13a43d5892d9 | ||
| github.com/smartcontractkit/chainlink-ccip/deployment v0.0.0-20260306193229-66635470b83d |
There was a problem hiding this comment.
devenv/go.mod bumps github.com/smartcontractkit/chainlink-ccip and .../deployment to commit 20260306193229-66635470b83d, but the same module file still requires github.com/smartcontractkit/chainlink-ccip/ccv/chains/evm at v0.0.0-20260116092715-df6a64d4bf00 (used by devenv/chainimpl/*/monitoring.go). This mixes CCIP components from different commits and can lead to incompatible gobindings/APIs. Align the ccv/chains/evm version with the bumped CCIP commit (or explain why it must remain pinned).
No description provided.