-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
feat(cli/tools/upgrade): canary support #8476
Conversation
d8321ae
to
312b5dd
Compare
Arg::with_name("canary") | ||
.long("canary") | ||
.help("Upgrade to canary builds"), |
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.
IMO this arg should take optional value that allows to specify concrete commit
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.
with the current state of the PR, that would be passed through version
@crowlKats Can you rebase and add a test? |
@lucacasonato sure; but currently all upgrade tests seem to be disabled because flaky |
@crowlKats could you uncomment them and try to make them pass? |
# Conflicts: # cli/main.rs # cli/tools/upgrade.rs
not really sure what's going on... it isnt doing anything special... |
Description of current behaviour of upgrade with this PR:
these rules of course can be bypassed by |
The behavior looks good to me. It seems the CI failure is unrelated. |
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.
LGTM - thanks @crowlKats!
No description provided.