New consolidated CLI #786
New consolidated CLI #786
Conversation
initial modularization refactor of the data update process
replace existing CLI interface with new consolidated, click-based CLI
|
@buckley-w-david Thanks for your patience - I just resolved the slight merge conflict from the couple PRs that got merged before this. I'll review this shortly. Also, are you up for joining a Slack channel 18F/TTS runs, with a few other relevant folks, about this work? If so, do you want to send a quick email to [email protected] to share your work email with me? |
|
Sounds good, I sent you an email. I urge caution on actually merging in this change, it's fairly large and interfaces with large untested portions of the code, so we should be quite careful with it. The tests included strictly only test the CLI itself |
|
Yeah, that's part of the delay -- it needs some dedicated human testing. |
|
Do not remember closing this one, re-opening in case the features are still wanted |
As per #783, this PR proposes a replacement to the current CLI, with the interface outlined therein, with a change that the command is
pulseinstead ofdata.