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

Correct some typos #142

Merged
merged 1 commit into from
Apr 21, 2023
Merged

Correct some typos #142

merged 1 commit into from
Apr 21, 2023

Conversation

goggle
Copy link
Contributor

@goggle goggle commented Apr 20, 2023

Corrects some typos in the repository.

Copy link
Owner

@tompollard tompollard left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks! Please could you correct the napoleaon before we merge?

@@ -46,7 +46,7 @@ reStructuredText <http://sphinx.pocoo.org/rest.html>`__.
The documentation follows the
`NumPy Docstring Standard <https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt>`__,
which are parsed using the
`napolean extension for sphinx <http://www.sphinx-doc.org/en/1.5.1/ext/napoleon.html>`.
`napoleaon extension for sphinx <http://www.sphinx-doc.org/en/1.5.1/ext/napoleon.html>`.
Copy link
Owner

Choose a reason for hiding this comment

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

The a sneaked into your fix! Please could you remove it (napoleaon -> napoleon)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm sorry, it has been corrected now.

@goggle goggle force-pushed the typos01 branch 2 times, most recently from 13b4d3c to 6c29b68 Compare April 21, 2023 00:30
Signed-off-by: Alexander Seiler <seileralex@gmail.com>
@tompollard
Copy link
Owner

Thanks @goggle!

@tompollard tompollard merged commit c8f2662 into tompollard:main Apr 21, 2023
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.

2 participants