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

Fix typos #829

Merged
merged 1 commit into from
Jul 31, 2021
Merged

Fix typos #829

merged 1 commit into from
Jul 31, 2021

Conversation

hectormz
Copy link
Contributor

This PR fixes some typos found in the codebase/documentation.

Typos were found via codespell and reviewed by a human, so hopefully the changes are appropriate!

@mwouts
Copy link
Owner

mwouts commented Jul 27, 2021

Thank you @hectormz for this. I will give a look soon (I'll have to make sure that the correction on the sample ipynb files were reported as well on the code / markdown representations).

Also, did you include the documentation in the spell check?

@hectormz
Copy link
Contributor Author

Hi @mwouts ! I'm sorry to put the burden of checking on you. It did run on the documentation, but only found a few clear typos.

Theoretically, the same typos were found in fixed in all the file formats, but it was a little harder for me to diff the .ipynb to check (hence jupytext)

@mwouts
Copy link
Owner

mwouts commented Jul 31, 2021

Thanks @hectormz! I see that all the test ran OK, so indeed the changes on the ipynb files and on their text representation are in line, we're good to go! Thanks again

@mwouts mwouts merged commit 704e6f2 into mwouts:master Jul 31, 2021
@mwouts mwouts added this to the 1.12.0 milestone Jul 31, 2021
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