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

Add --list-archives-hashes #600

Closed
wants to merge 4 commits into from
Closed

Add --list-archives-hashes #600

wants to merge 4 commits into from

Conversation

gperciva
Copy link
Member

No description provided.

@gperciva
Copy link
Member Author

There's a question of grammar here: --list-archives-hashes is wrong; it should end with archive-hashes. However, I didn't like this combination of options:

--list-archives
--list-archive-hashes

Despite the grammar, I think it's more clear if we used:

--list-archives
--list-archives-hashes

Or maybe --list-archives-hash? At least that doesn't have two plurals.

Another possibility would be to add an entirely separate option, such as perhaps --hashes or --use-hashes. Then the GUI could do

--list-archives --use-hashes

That would have the benefit of not adding a new command mode.

@gperciva gperciva marked this pull request as draft December 11, 2023 01:48
@gperciva gperciva closed this Dec 12, 2023
@gperciva gperciva deleted the list-archives-hashes branch December 12, 2023 00:54
@gperciva gperciva restored the list-archives-hashes branch December 12, 2023 00:54
@gperciva gperciva deleted the list-archives-hashes branch December 12, 2023 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant