-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
# Motivation The flag `-l` was ambiguous in some `bin/dfx-software-.*-version` commands, with several options having the same short form. There is a risk of "just changing the flag" but in a way that makes the flags inconsistent or confusing so I reviewed all these flags. This convention seems to make sense and is mostly consistent with what we already have: * `-l --latest` The latest release. * `-m --mainnet` The version deployed on mainnet. * `-p --pinned` The version in bin/versions.bash * `-v --version` A named release, such as "latest". Enables passing through a version from one command to the next by name rather than with lots of different flags. # Changes * Change the short form for "pinned" to not conflict. * Implement the above convention consistently. # Tests I have searched for usages of `dfx-software-.*-version` and found that all uses in `snsdemo` and `nns-dapp` use the long form, so changing the short flags will not affect CI. It may affect command line usage - that is where I personally use short flags most frequently.
- Loading branch information
Showing
4 changed files
with
7 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters