Skip to content

Conversation

@pavel-raykov
Copy link
Contributor

@pavel-raykov pavel-raykov commented Jan 19, 2026

core ref: d735a58b8a90eb705759661d9aa867c079980e02

@github-actions
Copy link

Metric del-old-consts main
Coverage 70.3% 69.6%

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the unused pkg/consts package from the codebase, as indicated by the ticket reference CRE-491. The package has been replaced by a copy in the common module, making this local version redundant.

Changes:

  • Deleted the entire pkg/consts/consts.go file containing contract names, method names, event names, and related constants
  • Updated Go version from 1.25.3/1.25.4 to 1.25.5 across multiple modules
  • Updated various dependency versions to align with the latest chainlink-ccip core reference

Reviewed changes

Copilot reviewed 6 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/consts/consts.go Complete removal of the constants package
integration-tests/go.mod Go version bump and dependency updates
devenv/go.mod Go version bump and dependency updates
chains/solana/go.mod Dependency version updates
chains/solana/deployment/go.mod Dependency version updates
chains/evm/deployment/go.mod Go version bump and dependency updates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pavel-raykov pavel-raykov added this pull request to the merge queue Jan 21, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 21, 2026
@pavel-raykov pavel-raykov added this pull request to the merge queue Jan 21, 2026
Merged via the queue into main with commit 85accaf Jan 21, 2026
54 of 56 checks passed
@pavel-raykov pavel-raykov deleted the del-old-consts branch January 21, 2026 16: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.

3 participants