Skip to content

Fix NumPy capitalisation in docs. - #8144

Merged
sklam merged 2 commits into
numba:mainfrom
stuartarchibald:doc/numpy_spelling
Jun 21, 2022
Merged

sklam merged 2 commits into
numba:mainfrom
stuartarchibald:doc/numpy_spelling

Conversation

@stuartarchibald

Copy link
Copy Markdown
Contributor

As title. NumPy is written as "NumPy".

As title. NumPy is written as "NumPy".
@stuartarchibald
stuartarchibald marked this pull request as ready for review June 10, 2022 14:30

@esc esc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I used git grep in the docs subdirectory and only found the following, in a source code comment:

https://github.com/numba/numba/blob/main/docs/source/user/cfunc.rst?plain=1#L175

@esc esc added 4 - Waiting on author Waiting for author to respond to review and removed 3 - Ready for Review labels Jun 16, 2022
Fix up a "numpy" that was missed in the docs.
@stuartarchibald stuartarchibald added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 4 - Waiting on author Waiting for author to respond to review labels Jun 21, 2022
@esc

esc commented Jun 21, 2022

Copy link
Copy Markdown
Member

Thank you for the patch!

@stuartarchibald

Copy link
Copy Markdown
Contributor Author

I used git grep in the docs subdirectory and only found the following, in a source code comment:

https://github.com/numba/numba/blob/main/docs/source/user/cfunc.rst?plain=1#L175

Thanks, fixed in 146e72d

@esc esc added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Jun 21, 2022
@esc esc added this to the Numba 0.56 RC milestone Jun 21, 2022
@sklam
sklam merged commit 9a7820c into numba:main Jun 21, 2022
@esc esc mentioned this pull request Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Review and testing done, is ready to merge Effort - short Short size effort needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants