gleitz / howdoi

instant coding answers via the command line
http://blog.gleitzman.com/post/43330157197/howdoi-instant-coding-answers-via-the-command-line
MIT License
10.56k stars 867 forks source link

rough draft: attempt to make `keep` an extra #458

Open justin-f-perez opened 2 years ago

justin-f-perez commented 2 years ago

attempt to make keep an extra to put it on the PATH when installing via pipx/homebrew

For anyone who might have time to take up the mantle of keeping keep on the PATH here (see #425 for context):

I missed a quote. Let's pretend it was intentional to keep this rush-job from getting merged in ;) I have no experience with setuptools and unfortunately don't have more time to tinker- just stumbled across this potential solution and thought I should share

Of particular interest in #425 thread, last comment: as mentioned, consider use case where user opts not to install keep.

How to test:

I think you should be able to clone this branch, fix the quote, and pipx install .[keep]

Not sure how to test w/ brew

@gleitz feel free to close if this just adds noise to your repo i wont be offended :)

Please provide detailed instructions for testing your changes locally, including expected response/behavior.

Pull Request checklist:

Known bugs (if any):

If there are bugs in your current changes you can still open the PR and mention the bugs you found. Propose further changes that can help fix bugs in your current changes.

gleitz commented 2 years ago

This is a valiant effort, thank you