-
Notifications
You must be signed in to change notification settings - Fork 90
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 spelling mistakes #195
Conversation
This commit fixes a bunch of spelling mistakes, found and fixed by codespell v2.2.1. All fixes are checked by hand to be actually a fix for an actually misspelled word.
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@quaquel Quick update on this effort, and a few questions:
Furthermore I looked into adding codespell to the pre-commit configuration. Unfortunately, it's still to too buggy for that, and mainly blocked by codespell-project/codespell#2138 (images in Jupyter Notebooks get sometimes corrupted). Which is a shame, because it would be really useful catching mistakes in PRs. |
|
I threw out the spelling fixes in the Vensim models and the weird Juypter notebook image issue. I also resolved conflicts. This PR is now ready for review. Please squash when merging. |
This commit fixes a bunch of spelling mistakes, found and fixed by codespell v2.2.1. All fixes are checked by hand to be actually a fix for an actually misspelled word.
Still in draft because I need to check a few special cases and the Jupyter Notebooks.
Once those are fixed or added to a ignore list/file, I plan to also add codespell to the pre-commit CI to catch future errors.