bump sui binding#560
Conversation
🦋 Changeset detectedLatest commit: dc67a3c The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
product-security-plaid-production
Bot
requested a review
from gustavogama-cll
|
👋 huangzhen1997, 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
This PR updates Sui bindings by upgrading the Go version and updating various dependencies, including the chainlink-sui package. The main changes include updating the SUI CCIP entrypoint encoder constructor calls to match the new API.
- Go version upgraded from 1.24.4 to 1.24.5
- Multiple dependency version updates including chainlink-sui, aptos-go-sdk, and various other libraries
- Updated SUI CCIP entrypoint encoder constructor calls to include registry object parameter
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| go.mod | Updated Go version and numerous dependency versions including chainlink-sui bindings |
| engine/cld/legacy/cli/mcmsv2/mcms_v2.go | Updated SUI CCIP entrypoint encoder constructor calls to match new API signature |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jkongie
left a comment
jkongie
left a comment
There was a problem hiding this comment.
You will need to add to the changelog.
Please use pnpm changeset add with use a minor version bump
huangzhen1997
dismissed stale reviews from gustavogama-cll, rodrigombsoares, and tt-cll
via
8673dc8
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…kit/chainlink-deployments-framework into bump-chainlink-sui-version
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
huangzhen1997
requested review from
gustavogama-cll,
jkongie,
rodrigombsoares and
tt-cll
|
gustavogama-cll
dismissed
jkongie’s stale review
bumping the go version is required; we'll fix the other references later so we can merge this and unblock other teams
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## chainlink-deployments-framework@0.63.0 ### Minor Changes - [#560](#560) [`ed679a7`](ed679a7) Thanks [@huangzhen1997](https://github.com/huangzhen1997)! - bump sui binding version ### Patch Changes - [#561](#561) [`280ce37`](280ce37) Thanks [@gustavogama-cll](https://github.com/gustavogama-cll)! - fix: computation of the txNonce attribute in the upf converter --------- Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
## Desc Addressing #560 Update Sui MCMS lib
Bump SUI version to fix CI in CLD when bump core version