Conversation
There was a problem hiding this comment.
Always nice to see code cleanup, @JMGilbert 👍
If this removes all of cli, then I think we can remove the click package as a dependency in README.md and requirements.txt. I'd consider including those changes in this PR and maybe noting that we're dropping the CLI in CHANGELOG... I don't know if anyone is actually using it or would look for it again in the future. I feel like y'all would be better judges of that over me.
@kemccusker, @JMGilbert I see this takes care of fair. I also had a note about possibly removing diagnostics. Is that something you'd want to do here or is that a change for another time?
That's my only thought. I think this looks good-to-go otherwise.
|
@brews we will take care of cleaning up |
Deleting unused files