OCHA-DAP / hdx-cli-toolkit

A commandline tool for interacting with HDX with a view to doing bulk updates
MIT License
1 stars 0 forks source link

Refactor cli.py #9

Closed IanHopkinson closed 7 months ago

IanHopkinson commented 8 months ago

Currently cli.py contains quite a lot of implementation that should go in hdx_utilities.py with cli.py just handling argument wrangling and reporting - this will make implementation testing easier since the click peculiarities won't be in play.

IanHopkinson commented 8 months ago

Also it would make hdx-toolkit a useful developer tool since they could simply import from hdx_utilities.py for simple activities

IanHopkinson commented 7 months ago

This touches on #16 and #1 - once a refactor is done we can publish to PyPI and simplify the documentation around installation.

IanHopkinson commented 7 months ago

Fixed in #19