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

PEP 257 docstrings #177

Closed
ianthetechie opened this issue Feb 6, 2016 · 2 comments
Closed

PEP 257 docstrings #177

ianthetechie opened this issue Feb 6, 2016 · 2 comments
Labels

Comments

@ianthetechie
Copy link
Contributor

The coding standards document indicates that the code in this project should conform to PEP 8, which specifies PEP 257 docstrings. My pull request #176 changed the files that it touched. I would be happy to take care of converting the rest of the project if you like.

@timothycrosley
Copy link
Collaborator

Thanks for notifying me about this! I didn't even realize the spec called for double quotes, the code base should certainly be updated to match the defined coding standards :)

@ianthetechie
Copy link
Contributor Author

No worries. It's one of those things that didn't get solidified until later on anyways. I'm likewise a fan of single quotes everywhere else, since that's one less shift key press ;) Working on this right now, as well as adding unit tests for the functions added in my last PR.

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