Skip to content

Conversation

@bjlittle
Copy link
Member

@bjlittle bjlittle commented Mar 6, 2023

🚀 Pull Request

Description

This PR automates the detection of speeling spelling mistakes within the code base, as part of our pre-commit.ci workflow using codespell.

Also adds a .git-blame-ignore-revs to ensure that I don't get blamed for the original authors change, see here for further details.


Consult Iris pull request check list

@codecov
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (7b6fa46) 89.31% compared to head (f77bf16) 89.31%.

❗ Current head f77bf16 differs from pull request most recent head 3038bb2. Consider uploading reports for the commit 3038bb2 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5186   +/-   ##
=======================================
  Coverage   89.31%   89.31%           
=======================================
  Files          88       88           
  Lines       22279    22279           
  Branches     4881     4881           
=======================================
  Hits        19898    19898           
  Misses       1635     1635           
  Partials      746      746           
Impacted Files Coverage Δ
lib/iris/_concatenate.py 95.42% <ø> (ø)
lib/iris/_merge.py 97.88% <ø> (ø)
lib/iris/analysis/_area_weighted.py 86.89% <ø> (ø)
lib/iris/analysis/_interpolation.py 95.80% <ø> (ø)
lib/iris/analysis/_regrid.py 83.33% <ø> (ø)
lib/iris/analysis/cartography.py 86.41% <ø> (ø)
lib/iris/analysis/trajectory.py 94.44% <ø> (ø)
lib/iris/common/metadata.py 93.23% <ø> (ø)
lib/iris/common/resolve.py 92.88% <ø> (ø)
lib/iris/experimental/regrid.py 77.38% <ø> (ø)
... and 14 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@bjlittle bjlittle changed the title Codespell Adopt codespell Mar 6, 2023
@bjlittle
Copy link
Member Author

This PR, by it's nature, is prone to go stale very quickly, which is fine.

I'm only going to resolve conflicts once we're happy to merge.

@bjlittle
Copy link
Member Author

Conflicts resolved 👍

@bjlittle bjlittle force-pushed the codespell branch 2 times, most recently from 07bdbde to 4368d7b Compare March 28, 2023 09:04
@tkknight tkknight merged commit 2eac400 into SciTools:main Apr 18, 2023
@bjlittle bjlittle mentioned this pull request Apr 18, 2023
@bjlittle bjlittle deleted the codespell branch April 19, 2023 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants