search

pepkit / geofetch

Builds a PEP from SRA or GEO accessions
https://pep.databio.org/geofetch/
BSD 2-Clause "Simplified" License
46 stars 5 forks source link

Decouple Geofetcher and argparser #56

Closed khoroshevskyi closed 2 years ago

khoroshevskyi commented 2 years ago

The Geofetcher class is dependent on argparse, that provides limitations and issuses in future development.

Decoupling of Geofetcher and Argparser will solve this problem.

nsheff commented 2 years ago

We need to use Geofetcher in scenarios where we won't have a CLI.

nsheff commented 2 years ago

I don't believe this change was reflected in the changelog, was it?

khoroshevskyi commented 2 years ago

I don't believe this change was reflected in the changelog, was it?

In changelog, I have added arguments and features that were added to geofetch. Decoupling of geofetcher and argparser wasn't added to changelog.