Skip to content
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

Change CLI args for tags #389

Open
piegamesde opened this issue Jun 12, 2023 · 0 comments
Open

Change CLI args for tags #389

piegamesde opened this issue Jun 12, 2023 · 0 comments
Labels
enhancement New feature or request maybe May not happen or be useful

Comments

@piegamesde
Copy link
Contributor

Currently tags are parsed as positional arguments, before paths. As this might be ambiguous, the special argument :: is used to separate both, although it is optional. I am not a fan of this design for multiple reasons. Mainly, I am scared of accidentally mixing things up and then getting silent misbehavior of the application. But also the special :: argument is confusing, as it is similar but not identical to the -- convention which separates optional from positional arguments. (I don't know whether bupstash makes use of that convention already, if not then it probably should but that's another issue.)

Instead, I'd like to see tags added with the optional argument --tag, which can then be specified multiple times. This is more typing effort, but IMO it is worth the general improvement. Alternatively, one could have --tags with a comma separate list of tag values to save some typing, although this would only work for tags which don't contain a comma (i.e. most)

@andrewchambers andrewchambers added enhancement New feature or request maybe May not happen or be useful labels Jun 13, 2023
piegamesde added a commit to piegamesde/bupstash that referenced this issue Feb 3, 2024
Closes andrewchambers#389. Also implements a duplicates check, therefore fixes andrewchambers#396.
piegamesde added a commit to piegamesde/bupstash that referenced this issue Feb 3, 2024
Closes andrewchambers#389. Also implements a duplicates check, therefore fixes andrewchambers#396.
piegamesde added a commit to piegamesde/bupstash that referenced this issue Feb 3, 2024
Closes andrewchambers#389. Also implements a duplicates check, therefore fixes andrewchambers#396.
piegamesde added a commit to piegamesde/bupstash that referenced this issue Feb 3, 2024
Closes andrewchambers#389. Also implements a duplicates check, therefore fixes andrewchambers#396.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request maybe May not happen or be useful
Projects
None yet
Development

No branches or pull requests

2 participants