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

Refactoring idea: one taxaList and a flag for the include/exclude switch #10

Open
kbshimmyo opened this issue Jan 24, 2018 · 1 comment

Comments

@kbshimmyo
Copy link
Collaborator

This would clean up a lot of the validation if we didn't have separate -i taxaList and -e taxaList that we have to manually enforce the mutual exclusion for, and also the code for downloading the exclude list and include list separately.

Suggestion:
-t taxaList
-i|e without arguments.

Then validation consists of making sure that if -t is provided, -i|e must also be provided.

@gedankenstuecke
Copy link
Collaborator

Yes, that sounds good 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants