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

Automate spell check #657

Merged
merged 62 commits into from
Jun 25, 2019
Merged

Automate spell check #657

merged 62 commits into from
Jun 25, 2019

Conversation

guyer
Copy link
Member

@guyer guyer commented Jun 24, 2019

Uses Pyspelling

guyer added 30 commits June 18, 2019 16:27
Now that develop is gone, no need to merge master
Accommodate docstring format we adopted (from where?!?) ages ago
that has leading hyphens before each parameter description
@guyer guyer requested review from tkphd and wd15 June 24, 2019 20:17
@wd15
Copy link
Contributor

wd15 commented Jun 25, 2019

This is really awesome. Can you set it up to handle different formatting types in the docstring?

Copy link
Contributor

@tkphd tkphd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Best. PR. Ever.

@guyer guyer merged commit 86aaa5c into usnistgov:master Jun 25, 2019
@guyer guyer deleted the spellcheck branch June 25, 2019 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants