search

alex-rudakov / sphinx-argparse

Sphinx extension that automatically document argparse commands and options
MIT License
49 stars 40 forks source link

Fix most PEP-8 issues #13

Closed zyga closed 10 years ago

zyga commented 10 years ago

Hi.

I've fixed most of the PEP-8 issues I could find. The code (apart from tests) is now PEP-8 clean as reported by flake8. Tests still pass (tested 2.7 and 3.4). I'd be grateful if you'd merge that in. I plan on making some additional changes (new features) that I need for my project and this would help me a lot to share those back to you.

Best regards ZK

alex-rudakov commented 10 years ago

Hi,

thanks for fixes! Will not compile a new version of pypi package this time, as there is no new functionality.

I plan on making some additional changes

Email me if you need some help from me with this.

zyga commented 10 years ago

Thanks a lot for merging that!

My main goal is to improve output when sphinx generates man pages. So far I'm experimenting and adding support for manpages to my project (plainbox) and sphinx-argparse is already working very well.

I will improve documentation based on what I've learned. So far I plan to document the :nodefault: option.

Thanks ZK

2014-07-16 15:21 GMT+02:00 Alex Rudakov notifications@github.com:

Hi,

thanks for fixes! Will not compile a new version of pypi package this time, as there is no new functionality.

I plan on making some additional changes

Email me if you need some help from me with this.

— Reply to this email directly or view it on GitHub https://github.com/ribozz/sphinx-argparse/pull/13#issuecomment-49163810.