ci: update codespell skip list to remove duplicate entry.

[terabytesoftw]

Pull Request

• Breaking change (fix or feature that would cause existing functionality to change)
• Bugfix (non-breaking change that fixes an issue)
• CI/build configuration
• Documentation update
• New feature (non-breaking change that adds functionality)
• Refactoring (no functional changes)

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 72477bd0-e3f5-4f9a-8de8-fc2fa138e201

📥 Commits

Reviewing files that changed from the base of the PR and between 113b989 and d244a0e.

📒 Files selected for processing (2)

• .github/linters/.codespellrc
• CHANGELOG.md

📜 Recent review details

🔇 Additional comments (2)

.github/linters/.codespellrc (1)

2-2: 📐 Maintainability & Code Quality | ⚡ Quick win

Verify the skip-glob scope change.

tests/* no longer excludes nested test trees, and */composer.lock will stop skipping a root-level composer.lock if one exists. If those paths are still meant to stay out of codespell, this change will reintroduce CI noise.

CHANGELOG.md (1)

10-11: LGTM!

---

📝 Walkthrough

Summary by CodeRabbit

• Chores
  • Updated spelling-check ignore coverage to be more specific for test files and remove a duplicate skip entry.
• Documentation
  • Added a changelog note describing the CI spelling-check adjustment.

Walkthrough

Modifies the codespell skip pattern list in .github/linters/.codespellrec, removing the composer.lock entry and narrowing tests/** to tests/*, with a corresponding CHANGELOG.md entry documenting the CI configuration change.

Changes

Codespell Configuration Cleanup

Layer / File(s)	Summary
Skip pattern update and changelog entry .github/linters/.codespellrc, CHANGELOG.md	Removes the root composer.lock skip entry, replaces recursive tests/** with tests/*, and adds a changelog note about removing the duplicate skip entry.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• php-forge/baseline#2: Both PRs update .codespellrc skip patterns for test/composer files.
• php-forge/baseline#13: Both PRs modify the Codespell skip patterns around composer.lock/tests paths.

Poem

A rabbit hopped through lint and rule,
Trimmed a skip line, kept things cool. 🐇
No more dupes in composer's lock,
Tests just get a lighter knock,
Changelog says: "all's well, all's well!"

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is concise and accurately summarizes the CI codespell skip-list change.
Description check	✅ Passed	The description is related to the change and correctly marks it as a refactoring with no functional impact.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ci/update-codespell-skip-list

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}