Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor docstrings to use Napoleon syntax #37

Open
EliAndrewC opened this issue Sep 14, 2016 · 1 comment
Open

Refactor docstrings to use Napoleon syntax #37

EliAndrewC opened this issue Sep 14, 2016 · 1 comment
Labels

Comments

@EliAndrewC
Copy link
Contributor

Moved here from the original (now private) issue tracker.

We have a lot of docstrings (though we should really add more), and I really like the formatting examples for https://pypi.python.org/pypi/sphinxcontrib-napoleon so I think we should start using that. Seriously, check out how much nicer it looks than ReStructuredText.

This would also be a really good opportunity for a newcomer to div into the Sideboard codebase and see what it's doing, since it'll force you to read through and understand the docs!

@kitsuta
Copy link
Member

kitsuta commented Sep 15, 2016

Related to this, I wonder if we can configure PyCharm to use the Google or NumPy (I prefer Google) docstring format for when you're inserting docstrings. The docstring insertion is super handy and it'd be nice if it used the nicer format by default. I'll look into this briefly but Dom may know how to do it out of hand - it's apparently configured in the Python Integrated Tools plugin here: https://www.jetbrains.com/help/pycharm/2016.1/python-integrated-tools.html

kitsuta added a commit to magfest-archive/simple-rams-deploy that referenced this issue Sep 15, 2016
As mentioned in magfest/sideboard#37, we want to move towards better-looking docstrings. This helps that by making the 'doc string insertion' feature default to the Google docstring format. That means when someone types `"""` and hits Enter, they'll get a docstring template formatted Google-style.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants