Skip to content

🪲 [Fix]: Add .codespellrc to skip linter config directory#9

Merged
Marius Storhaug (MariusStorhaug) merged 1 commit intomainfrom
fix/add-codespellrc-skip-linters
Apr 5, 2026
Merged

🪲 [Fix]: Add .codespellrc to skip linter config directory#9
Marius Storhaug (MariusStorhaug) merged 1 commit intomainfrom
fix/add-codespellrc-skip-linters

Conversation

@MariusStorhaug
Copy link
Copy Markdown
Member

Context

The super-linter SPELL_CODESPELL check flags intentional misspellings in .github/linters/.textlintrc (e.g. environemnt, pacakge). These strings exist as linter configuration patterns and are not actual typos.

The GitHub, GoogleFonts, and NerdFonts repos already solved this by adding a .codespellrc file.

Changes

  • Added .github/linters/.codespellrc that tells codespell to skip the .github/linters directory and ignore the word afterall

Copilot AI review requested due to automatic review settings April 5, 2026 01:01
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

No Significant Changes Detected

This PR does not contain changes to files that would trigger a new release:

Path Description
src/** Module source code
README.md Documentation

Build, test, and publish stages will be skipped for this PR.

If you believe this is incorrect, please verify that your changes are in the correct locations.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a codespell configuration to prevent Super-Linter’s SPELL_CODESPELL check from flagging intentional “misspellings” inside linter configuration files under .github/linters.

Changes:

  • Added .github/linters/.codespellrc with a skip rule for .github/linters.
  • Added an ignore-words entry for afterall.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -0,0 +1,3 @@
[codespell]
skip = ./.github/linters
Copy link

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

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

codespell path matching can vary between emitting paths like .github/linters/... vs ./.github/linters/.... To make the skip rule more robust, consider dropping the leading ./ (or include both patterns) so the directory is reliably skipped regardless of how paths are reported.

Suggested change
skip = ./.github/linters
skip = .github/linters

Copilot uses AI. Check for mistakes.
@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit c385cfd into main Apr 5, 2026
21 checks passed
@MariusStorhaug Marius Storhaug (MariusStorhaug) deleted the fix/add-codespellrc-skip-linters branch April 5, 2026 01:03
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

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