Skip to content

CIP-8334 Minor cleanup in verifier code#428

Merged
makramkd merged 3 commits intomainfrom
verifiers-cleanup
Dec 5, 2025
Merged

CIP-8334 Minor cleanup in verifier code#428
makramkd merged 3 commits intomainfrom
verifiers-cleanup

Conversation

@mateusz-sekara
Copy link
Collaborator

@mateusz-sekara mateusz-sekara commented Dec 5, 2025

@mateusz-sekara mateusz-sekara marked this pull request as ready for review December 5, 2025 14:40
@mateusz-sekara mateusz-sekara requested review from a team and skudasov as code owners December 5, 2025 14:40
with:
repository: smartcontractkit/chainlink
ref: becbfb2fc0774c97f7168780563ac9090aa3e3f1
ref: 585838629ce2be9fcf3f0ae9d79b72de6b892f08
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a reason for bumping or just staying up to date w/ latest develop on chainlink?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Actually this looks like a non-develop branch, if you can't merge the chainlink PR in time make sure to re-bump after a merge (and keep that branch un-deleted on remote)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I need a fix in a chainlink repo, so I need

  • merge in chainlink repo
  • merge this branch
  • bump to the main version in chainlink repo and merge as well

Comment on lines -21 to -22
docker tag verifier:rc {{registry}}/chainlink-ccv-token-verifier:{{version}}-rc
docker push {{registry}}/chainlink-ccv-token-verifier:{{version}}-rc
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this no longer needed? If we're running in standalone we'll need these images no?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It breaks the build, so temporary removing it. It won't be needed until we actuallly need to deploy on staging

@makramkd makramkd added this pull request to the merge queue Dec 5, 2025
Merged via the queue into main with commit 301e250 Dec 5, 2025
14 checks passed
@makramkd makramkd deleted the verifiers-cleanup branch December 5, 2025 20:45
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