chore: Bump chainlink-canton#787
Conversation
|
👋 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! |
There was a problem hiding this comment.
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-cantonandgithub.com/smartcontractkit/go-damlto newer commits. - Add a decoder test covering
DeployExecutorParams.allowedFinalityConfigvariant 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.
|




No description provided.