Skip to content

chore: read the codespell settings from pyproject.toml - #4213

Open
oresttokovenko wants to merge 1 commit into
apache:mainfrom
oresttokovenko:codespell-config-in-pyproject
Open

oresttokovenko wants to merge 1 commit into
apache:mainfrom
oresttokovenko:codespell-config-in-pyproject

Conversation

@oresttokovenko

Copy link
Copy Markdown
Contributor

What

Move the codespell settings from .codespellrc into [tool.codespell] in pyproject.toml.

Why

Every other tool in this repository is configured in pyproject.toml. .codespellrc holds a single line, and the file only exists because codespell had nowhere else to go when it was added.

Tests

prek run codespell --all-files passes on a file containing BoundIn NotIn fo oT, the words on the ignore list, and fails on teh, so the pass is not vacuous. The same file fails when codespell runs from a directory where the config is not visible, which shows the section in pyproject.toml is what makes it pass.

@rambleraptor

Copy link
Copy Markdown
Collaborator

CI failures will be fixed by #4216

@Fokko Fokko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohh, this is nice @oresttokovenko Less is more

@Fokko
Fokko enabled auto-merge October 31, 2026 22:25
codespell reads [tool.codespell], so the separate .codespellrc file is one less thing at the root of the repository.
auto-merge was automatically disabled October 31, 2026 22:31

Head branch was pushed to by a user without write access

@oresttokovenko
oresttokovenko force-pushed the codespell-config-in-pyproject branch from bc00c24 to ee4c36d Compare October 31, 2026 22:31

This branch has not been deployed

No deployments
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.

3 participants

⌂ Proxy