Skip to content

[CI] Centralize codespell config; remove duplication#2749

Merged
jiayuasu merged 2 commits into
apache:masterfrom
jbampton:standardize-codespell-config
Mar 16, 2026
Merged

[CI] Centralize codespell config; remove duplication#2749
jiayuasu merged 2 commits into
apache:masterfrom
jbampton:standardize-codespell-config

Conversation

@jbampton
Copy link
Copy Markdown
Member

Did you read the Contributor Guide?

Is this PR related to a ticket?

  • No:
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

We run codespell with pre-commit seems we had two sets of configs

How was this patch tested?

With pre-commit

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@jbampton jbampton requested a review from jiayuasu as a code owner March 13, 2026 11:39
@jbampton jbampton added root pre-commit Pull requests that update pre_commit code labels Mar 13, 2026
@jbampton jbampton marked this pull request as draft March 13, 2026 11:44
@jbampton jbampton marked this pull request as ready for review March 13, 2026 12:08
@jiayuasu jiayuasu merged commit d5c8e16 into apache:master Mar 16, 2026
9 checks passed
@jbampton jbampton deleted the standardize-codespell-config branch March 16, 2026 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pre-commit Pull requests that update pre_commit code root

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants