-
Notifications
You must be signed in to change notification settings - Fork 255
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
Normalizing twarc2 commands #608
Comments
I'm in favour of this, especially if it means we stop guessing or inferring what was intended. This is probably enough of a breaking change that we'd want to have at least a major version bump, and ideally some nice managing of errors that we can tell people about the new syntax. This would also let us standardise the handling of usernames, so hopefully we don't have to deal with other instances like #568 and #542. |
Maybe all commands that do bulk operations on an input text file vs a single argument should be like |
I like the ID of a bulk subcommand, I can definitely see how that would contribute to consistency: anything that takes an input file is a That also gives us a nicer migration path because we can decouple creating new (more consistent) commands under bulk from removing the old commands, allowing for some deprecation warnings or similar. |
I'm finding that some command line commands are starting to diverge, and become inconsistent. This issue is just for investigating this and coordinating some changes i think we should make. Maybe some breaking changes? It would be great to align all the commands so that they have consistent commonalities and documentation and examples.
To begin with:
name-or-id
pattern is way better than guessing, it should be the same across the others.name or id
pattern maybename-or-id
pattern for supporting commands specifying user IDs and usernamesThe text was updated successfully, but these errors were encountered: