feat(linter): add codespell linter #6256
Conversation
ideaship
requested review from
Kurt-von-Laven,
bdovaz,
echoix and
nvuillam
as code owners
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
ideaship
force-pushed
the
rl_add_codespell
branch
from
7acabdc to
3f7cc67
Compare
ideaship
force-pushed
the
rl_add_codespell
branch
from
3f7cc67 to
475b23f
Compare
|
This pull request has been automatically marked as stale because it has not had recent activity. If you think this pull request should stay open, please remove the |
github-actions
bot
added
the
O: stale 🤖
nvuillam
removed
the
O: stale 🤖
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| @@ -0,0 +1,2 @@ | |||
| abandonded Dockefile | |||
| abberivated acadamy | |||
There was a problem hiding this comment.
Bug: Autofix Test File: Misspellings Remain Unfixed
The spell_fix_1.md test file contains the same misspellings as spell_bad_1.md, but should contain the corrected version. Since codespell has cli_lint_fix_arg_name: --write-changes defined, autofix tests expect this file to show the corrected output after applying fixes. It should contain "abandoned Dockerfile\nabbreviated academy" to match the corrections in spell_good_1.md.
|
Can the conflicts be solved? |
ideaship
force-pushed
the
rl_add_codespell
branch
from
475b23f to
d16a1ac
Compare
| - .jsonc | ||
| - .json5 | ||
| - .yaml | ||
| - .yml |
There was a problem hiding this comment.
Bug: Restrictive file extensions prevent checking code
The file_extensions list only includes documentation and data files, preventing codespell from checking source code files (like .py, .js, etc.) as intended by its description and purpose. Consider adding * or a broader list of code extensions to ensure it runs on the codebase.
Add the codespell linter which uses a dictionary of common misspellings and ignores unknown words. Signed-off-by: Roger Luethi <roger.luethi@bisdn.de>
ideaship
force-pushed
the
rl_add_codespell
branch
from
d16a1ac to
d4e36a0
Compare
| # renovate: datasource=pypi depName=codespell | ||
| ARG PIP_CODESPELL_VERSION=2.4.1 | ||
| pip: | ||
| - codespell==${PIP_CODESPELL_VERSION} |
There was a problem hiding this comment.
Bug: Missing lint_all_other_linters_files property for codespell
The codespell linter definition is missing the lint_all_other_linters_files property. Since the SPELL descriptor sets lint_all_other_linters_files: true at the descriptor level, codespell will inherit this value and lint all files from all other linters instead of only the specified file extensions. All other spell linters in this descriptor (proselint, vale, and lychee) explicitly set lint_all_other_linters_files: false to override the descriptor-level setting. This causes codespell to operate on an unexpectedly large set of files beyond its intended scope.
|
@nvuillam seems ready |
3 participants
Add the codespell linter which uses a dictionary of common misspellings and ignores unknown words.
This is an updated version of #4143 which was closed as stale and could not be reopened by the maintainer.
Fixes #6255
Proposed Changes
Add the codespell linter which uses a dictionary of common misspellings and ignores unknown words.
Readiness Checklist
Author/Contributor
Reviewing Maintainer
breakingif this is a large fundamental changeautomation,bug,documentation,enhancement,infrastructure, orperformanceNote
Introduces the codespell linter across MegaLinter (descriptors, schemas, matrices, flavors, Docker builds) with docs, tests, and version/link updates.
SPELL_CODESPELL(codespell)megalinter/descriptors/spell.megalinter-descriptor.ymlwith config, examples, and install.megalinter-configuration.jsonschema.json,megalinter-custom-flavor.jsonschema.json) and flavors lists (all_flavors.json, per-flavorflavor.json)..automation/generated/linters_matrix.json) and versions/links (linter-versions.json,linter-links-previews.json).linters/spell_codespell/Dockerfile; list in docsdocs/standalone-linters.md.PIP_CODESPELL_VERSION, venv, PATH entries).megalinter/tests/.../spell_codespell_test.pyand.automation/test/spell_codespell/*..mega-linter.yml(disable errors),.cspell.jsonadditions, changelog entry.Written by Cursor Bugbot for commit 16b8171. This will update automatically on new commits. Configure here.