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

Consider merging all tools into one tool #7

Closed
tseemann opened this issue Jun 29, 2017 · 3 comments
Closed

Consider merging all tools into one tool #7

tseemann opened this issue Jun 29, 2017 · 3 comments

Comments

@tseemann
Copy link

Currently you have

analysisGet     assemblyGet     enaDataGet     enaGroupGet     readGet     sequenceGet

I think it would be much better to have a single executable tool ena-get (or similar) where the type of object could specified (or auto-predicted) ? Or use subcommands like samtools and other modern tools. (Note that argparse supports sub-commands.)

Is this possible?

@nicsilvester
Copy link
Contributor

You only need enaDataGet and enaGroupGet. The others were left from when it was being built, but yes we can remove the main method from them.
For now enaDataGet and enaGroupGet still need to be kept separate to keep the command line usage as simple as possible. We can revisit subcommands later

@tseemann
Copy link
Author

If they aren't needed they should be removed, or turned into modules, or at least made non-executable?

We are attempting to package this into Homebrew.

@tseemann
Copy link
Author

Ok, i see you have fixed this now. Thank you.

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

No branches or pull requests

2 participants