Skip to content

chore: Bump chainlink-canton#787

Merged
JohnChangUK merged 3 commits into
mainfrom
bump-canton
Jun 15, 2026
Merged

chore: Bump chainlink-canton#787
JohnChangUK merged 3 commits into
mainfrom
bump-canton

Conversation

@JohnChangUK

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 10, 2026 23:31
@JohnChangUK JohnChangUK requested a review from a team as a code owner June 10, 2026 23:31
@github-actions

Copy link
Copy Markdown

👋 JohnChangUK, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Canton bindings dependencies and adds a regression test to ensure the Canton decoder correctly handles a core.FinalityConfig Daml variant when decoding CCIPFactory::DeployExecutor operation data, including round-trip (decode → re-encode) correctness.

Changes:

  • Bump github.com/smartcontractkit/chainlink-canton and github.com/smartcontractkit/go-daml to newer commits.
  • Add a decoder test covering DeployExecutorParams.allowedFinalityConfig variant decoding and strict round-trip selection behavior.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
sdk/canton/decoder_test.go Adds a regression test for decoding and round-tripping DeployExecutorParams with a Daml variant field.
go.mod Updates required versions of chainlink-canton and go-daml.
go.sum Refreshes module checksums for the bumped dependencies.

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

stackman27
stackman27 previously approved these changes Jun 10, 2026
@JohnChangUK JohnChangUK enabled auto-merge (squash) June 10, 2026 23:34
@JohnChangUK JohnChangUK changed the title Bump chainlink-canton chore: Bump chainlink-canton Jun 10, 2026
Copilot AI review requested due to automatic review settings June 10, 2026 23:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated no new comments.

ecPablo
ecPablo previously approved these changes Jun 11, 2026
Copilot AI review requested due to automatic review settings June 13, 2026 10:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated no new comments.

@cl-sonarqube-production

Copy link
Copy Markdown

@JohnChangUK JohnChangUK requested a review from ecPablo June 13, 2026 17:18
@JohnChangUK JohnChangUK merged commit cc35ca0 into main Jun 15, 2026
22 checks passed
@JohnChangUK JohnChangUK deleted the bump-canton branch June 15, 2026 17:26
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.

4 participants