Skip to content

Conversation

bjlittle
Copy link
Member

@bjlittle bjlittle commented Jun 3, 2024

🚀 Pull Request

Description

Fix the codespell failures raised by pre-commit.ci for the latest version of codespell.

See #5980

Note that, this PR targets the base branch of #5980, rather than main.


Consult Iris pull request check list


Add any of the below labels to trigger actions on this PR:

  • benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts

Copy link

codecov bot commented Jun 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.78%. Comparing base (e100a3c) to head (4d85311).

Current head 4d85311 differs from pull request most recent head 94d57cb

Please upload reports for the commit 94d57cb to get more accurate results.

Additional details and impacted files
@@                       Coverage Diff                       @@
##           pre-commit-ci-update-config    #5985      +/-   ##
===============================================================
+ Coverage                        89.75%   89.78%   +0.03%     
===============================================================
  Files                               90       93       +3     
  Lines                            22929    23017      +88     
  Branches                          5020     5020              
===============================================================
+ Hits                             20580    20666      +86     
- Misses                            1618     1620       +2     
  Partials                           731      731              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d879dcb to e100a3c Compare June 3, 2024 17:57
@pp-mo
Copy link
Member

pp-mo commented Jun 4, 2024

Sorry @bjlittle
there's a problem now crept in with needing a rebase, somehow.
Maybe another pre-commit update on the original ?

@bjlittle bjlittle force-pushed the pre-commit-ci-update-config branch from 4d85311 to 94d57cb Compare June 4, 2024 09:41
@bjlittle
Copy link
Member Author

bjlittle commented Jun 4, 2024

@pp-mo Resolved the conflict. The pre-commit.ci bot had updated the already open PR, so I required to rebase.

Done. Over to you ...

@pp-mo pp-mo merged commit dbb8296 into SciTools:pre-commit-ci-update-config Jun 4, 2024
pp-mo pushed a commit that referenced this pull request Jun 4, 2024
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.4.4 → v0.4.7](astral-sh/ruff-pre-commit@v0.4.4...v0.4.7)
- [github.com/codespell-project/codespell: v2.2.6 → v2.3.0](codespell-project/codespell@v2.2.6...v2.3.0)

* fix codespell failures (#5985)

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Bill Little <bill.james.little@gmail.com>
@bjlittle bjlittle deleted the pre-commit-ci-update-config branch September 22, 2025 12:19
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