search

hayd / pep8radius

PEP8 clean only the parts of the files touched since the last commit, a previous commit or (the merge-base of) a branch.
MIT License
183 stars 9 forks source link

Suggestion - new CLI library #50

Closed schodge closed 10 years ago

schodge commented 10 years ago

https://github.com/mitsuhiko/click

Since you're looking for a challenge. :)

hayd commented 10 years ago

Don't understand this issue! What is click / what are you suggesting here?

(Is it a way to easily write CLI applications? .... cos we already have it working with argparse!)

schodge commented 10 years ago

It's a newer and shinier library! I suggest mostly in jest but it does look interesting and may lead to cleaner code than argparse. It also seems to handle py 2 vs 3 differences automatically (well, some of them). Worth 5 mins scrolling through the docs to see if anything jumps out at you.

Non-ideal input device. Please excuse typos. On May 27, 2014 12:09 PM, "Andy Hayden" notifications@github.com wrote:

Don't understand this issue! What is click / what are you suggesting here?

(Is it a way to easily write CLI applications? .... cos we already have it working with argparse!)

— Reply to this email directly or view it on GitHubhttps://github.com/hayd/pep8radius/issues/50#issuecomment-44321116 .

hayd commented 10 years ago

I had a look, and does offer a slightly cleaner syntax. However, I actually just copy and pasted from autopep8 (as shared pretty much all args) so using this wouldn't have saved time. Also, as we pass on options to autopep8 and docformatter so this may not even work!

As we already have the argparsing stuff working I don't want to touch it! :)