search

sgkit-dev / bio2zarr

Convert bioinformatics file formats to Zarr
Apache License 2.0
24 stars 6 forks source link

Replace ``python -m bio2zarr vcf2zarr`` with ``python -m bio2zarr.vcf2zarr``? #203

Open jeromekelleher opened 3 months ago

jeromekelleher commented 3 months ago

I.e., make each command a subpackage, and give those subpackages an __main__ definition. This seems like a more rational approach to organising things, but it does mean that encapsulating all the CLI functionality in the cli.py module as we do now (which has a lot of advantages) may not be possible.

jeromekelleher commented 3 months ago

Note: if we do, get rid of the note on installation.md linking to this issue.

benjeffery commented 3 months ago

I feel like the space is more natural than the . but it's not a strongly held opinion. Nice to have all the cli in one file too.