Skip to content

Verified vault build: use stellar-cli 28.1.0 - #65

Merged
M0nsxx merged 1 commit into
mainfrom
fix/vault-build-cli-28
Nov 9, 2026
Merged

M0nsxx merged 1 commit into
mainfrom
fix/vault-build-cli-28

Conversation

@Eras256

@Eras256 Eras256 commented Nov 9, 2026

Copy link
Copy Markdown
Collaborator

The first nirium-vault-v0.1.0 tag run failed at the build step:

error: soroban-sdk requires stellar-cli v25.2.0+ to build a contract

Run: https://github.com/nirium-protocol/nirium-core/actions/runs/42677296301

The pinned workflow (stellar-expert/soroban-build-workflow v25.1.0) installs stellar-cli 25.1.0, and our contracts use soroban-sdk 29.0.1. There is no newer release of that workflow, so this pins its main commit 6b16426 (4 Oct 2026), which installs stellar-cli 28.1.0.

I read the whole diff between the old pin and the new one. It's two lines: the CLI version (25.1.0 to 28.1.0) and actions/checkout v4 to v6. Nothing else changed.

The failed run published no release and registered nothing, so after this merges the same tag can be recreated on the new main.

🤖 Generated with Claude Code

The first nirium-vault-v0.1.0 run failed: soroban-sdk 29 refuses to build
with stellar-cli 25.1.0, which the pinned v25.1.0 workflow uses. Pin the
workflow main commit 6b16426 instead (stellar-cli 28.1.0). The only other
change upstream since v25.1.0 is actions/checkout v4 to v6.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@M0nsxx M0nsxx left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed. The first vault tag run failed because the pinned workflow installs stellar-cli 25.1.0 and soroban-sdk 29 needs 25.2.0+. This pins the workflow's main commit (stellar-cli 28.1.0). The upstream diff is only the CLI version and checkout v4 to v6. Approving.

@M0nsxx
M0nsxx merged commit 8489481 into main Nov 9, 2026
4 of 6 checks passed
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