feat(cli)!: equalize short flags (-p always project; booleans go long) - #26
Merged
Conversation
…flag Equalizes the project short flag across all commands. Previously -p was --project in most commands but --parent in list/mine/normalize (project was -P there) and --pat in config. Now -p is always --project, and -P is the per-command secondary: --parent (list/mine/normalize), --priority (create/update), --pat (config). BREAKING: scripts using -p/-P on list, mine, normalize or config change meaning.
Frees the short alias held by a boolean so the value-taking --top gets it, matching --top -n on list/mine/runs. BREAKING: 'normalize -n' now means --top instead of --dry-run.
…hort) Applies the rule that short aliases are reserved for value-taking parameters. BREAKING: 'config -l' and 'pr-list -m' are no longer recognized; use --login / --mine.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Equalize option short flags
Makes short aliases consistent and reserves them for value-taking parameters.
-pis always--projectPreviously
-pwas--projectin most commands but--parentinlist/mine/normalize(project was-Pthere) and--patinconfig. Now-pis always--project, and the uppercase-Pis each command's secondary flag — mnemonically a "P" word:--parent(list/mine/normalize),--priority(create/update),--pat(config).Booleans yield their short to value parameters
normalize --dry-runis now long-only;-nmaps to--top, matching--top -non list/mine/runs.config --loginandpr-list --mineare now long-only.After this, no boolean holds a short — every
-xpoints to a value-taking parameter.Scripts using
-p/-Ponlist/mine/normalize/config,-nonnormalize, or-l/-monconfig/pr-listchange meaning or are no longer recognized. Warrants a major version bump.README updated (option tables, examples, and the
-p/-Pnote).