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

feat!: enhance annotator CLI options #446

Closed
wants to merge 9 commits into from
Closed

Conversation

jsstevenson
Copy link
Contributor

@jsstevenson jsstevenson commented Aug 30, 2024

  • Make the input VCF path an argument rather than an option. I think this better matches the intent of the tool (and it was the only required option, so might as well make it an arg, right?). Left it at nargs=1 but I think there's a case for making it nargs=-1.
  • pathlike args should be click.Path/pathlib.Path. Add additional configurations, like the input file must exist and the output files must be writable. Didn't make changes further up the stack because I wanted to confine this PR to just the Click interface.
  • remove some superfluous/inactive configs
  • Touch up help text

@jsstevenson jsstevenson closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant