Conversation
There was a problem hiding this comment.
Pull request overview
Updates the build/devenv module’s Chainlink CCIP-related dependencies and adapts the devenv environment config generation to match updated changeset inputs.
Changes:
- Bumped
github.com/smartcontractkit/chainlink-ccipsubmodules used by devenv (deployment + ccv/chains/evm) to a newer pseudo-version. - Updated devenv’s aggregator config generation to pass
Topology(and removedChainSelectors) to the CCIP changeset input. - Removed
ChainSelectorsfrom the indexer config changeset input to match the updated dependency API.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| build/devenv/go.sum | Updated checksums for newer chainlink-ccip submodule pseudo-versions. |
| build/devenv/go.mod | Bumped chainlink-ccip submodule versions used by devenv. |
| build/devenv/environment.go | Adjusted changeset input wiring for aggregator/indexer config generation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -17,8 +17,8 @@ require ( | |||
| github.com/rs/zerolog v1.34.0 | |||
| github.com/smartcontractkit/chain-selectors v1.0.97 | |||
| github.com/smartcontractkit/chainlink-ccip v0.1.1-solana.0.20260319175550-83cf59fe6839 | |||
There was a problem hiding this comment.
The PR title suggests a general "bump cl-ccip", but go.mod only updates the chainlink-ccip submodules (deployment + ccv/chains/evm) while the root github.com/smartcontractkit/chainlink-ccip requirement remains pinned to the older pseudo-version. This can result in multiple different commits of the same repo being pulled in for different module paths, which can be confusing and occasionally lead to subtle incompatibilities. Consider bumping the root chainlink-ccip module to the same commit (or add a short note explaining why the root module is intentionally kept back).
| github.com/smartcontractkit/chainlink-ccip v0.1.1-solana.0.20260319175550-83cf59fe6839 | |
| github.com/smartcontractkit/chainlink-ccip v0.0.0-20260323194646-82b60f38ec35 |
|
Code coverage report:
|
No description provided.