-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI: add check for explicit spl-token-cli version #34430
CI: add check for explicit spl-token-cli version #34430
Conversation
Sorry about the premature review request. Just appeasing shellcheck |
b81c375
to
d907d61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great! r+ ci
cc/ @yihau
Backports to the stable branch are to be avoided unless absolutely necessary for fixing bugs, security issues, and perf regressions. Changes intended for backport should be structured such that a minimum effective diff can be committed separately from any refactoring, plumbing, cleanup, etc that are not strictly necessary to achieve the goal. Any of the latter should go only into master and ride the normal stabilization schedule. |
Backports to the beta branch are to be avoided unless absolutely necessary for fixing bugs, security issues, and perf regressions. Changes intended for backport should be structured such that a minimum effective diff can be committed separately from any refactoring, plumbing, cleanup, etc that are not strictly necessary to achieve the goal. Any of the latter should go only into master and ride the normal stabilization schedule. Exceptions include CI/metrics changes, CLI improvements and documentation updates on a case by case basis. |
Just confirming the expected process... After a new minor version branch is created, PRs against the new-stable will fail until we update scripts/spl-token-cli-version.sh in that new-stable branch to pin a version here: If that's correct then this won't break any of the existing processes. |
Yes, that's correct. Thanks for assessing, @willhickey ! |
@t-nelson , do you mind re-approving? I can't approve Will's release.md change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🫶
Thanks @yihau !! |
* Add script to specify spl-token-cli version * Add check for spl-token-cli version to CI * Add minor release branch cleanup instruction to unblock CI on the new stable branch --------- Co-authored-by: Will Hickey <will.hickey@solana.com> (cherry picked from commit e6e191f)
* Add script to specify spl-token-cli version * Add check for spl-token-cli version to CI * Add minor release branch cleanup instruction to unblock CI on the new stable branch --------- Co-authored-by: Will Hickey <will.hickey@solana.com> (cherry picked from commit e6e191f)
…34430) (#34465) CI: add check for explicit spl-token-cli version (#34430) * Add script to specify spl-token-cli version * Add check for spl-token-cli version to CI * Add minor release branch cleanup instruction to unblock CI on the new stable branch --------- Co-authored-by: Will Hickey <will.hickey@solana.com> (cherry picked from commit e6e191f) Co-authored-by: Tyera <tyera@solana.com>
…34430) (#34464) * CI: add check for explicit spl-token-cli version (#34430) * Add script to specify spl-token-cli version * Add check for spl-token-cli version to CI * Add minor release branch cleanup instruction to unblock CI on the new stable branch --------- Co-authored-by: Will Hickey <will.hickey@solana.com> (cherry picked from commit e6e191f) * Set spl-token-cli version for installs --------- Co-authored-by: Tyera <tyera@solana.com>
Problem
If https://github.com/solana-labs/solana-program-library releases a new version of
spl-token-cli
that depends on the beta branch, this cargo-install-all command will likely break, due to rust-version and various other mismatches.Summary of Changes
Require the stable channel to set an explicit
$splTokenCliVersion
Check that this is set in CI
Use
$splTokenCliVersion
to set a--version
for the spl-token-clicargo install
commandWill fix #34429, with backports and (on v1.16) variable population