-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Clean up docstrings to PEP257 specs and enable CI linting with pep257 #1474
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
Comments
It's no longer blocked and open for everyone who wants to tackle this. |
@agjohnson, @gregmuellegger do you have an overview over which parts of this are still open (and maybe update the issue check list)? I see that |
If it weren’t obvious from the flurry of PRs, I’m planning to work on this. I’ll try not to have too many PRs open at once, so I don’t spam the repo. :-) |
@alexwlchan this is great, thanks for the help cleaning up! i'll keep up with the cleanup efforts so that things don't go stale. So far, things are looking good -- small PRs is a good way to not clobber some of the other work. |
Once #1471 addresses initial linting errors, we will continue linting clean up by targetting errors produced by pep257 linting. Similar to #1471, address docstring cleanup efforts in small chunks, which will be easier to merge into other work and will not create broad changes that are difficult to merge. This will encourage the clean up effort to be quick, and will not block other work.
Work addressing clean up efforts can reference this ticket.
Applications that may cause linting errors due to pep257 failures:
refs #1424
The text was updated successfully, but these errors were encountered: