← Back to issue list

static: add codespell excludes for .direnv

View original Github issue

Metadata

Project
snapcraft
Number
#3028
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #3028 added .direnv to codespell's static-check exclusions; merged as commit e6c3f09af.

Resolution: closed resolved

Reason: PR #3028 was merged after approval, preventing codespell from checking the direnv directory. Git history records the resolving change as commit e6c3f09af.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #3028 added .direnv to codespell's static-check exclusions; merged as commit e6c3f09af.
stealth/space-bunny-alpha — closed resolved Merged PR #3028 (commit e6c3f09af), adding .direnv to the codespell exclusions so static checks ignore the directory-environment path.
qwen/qwen3.6-35b-a3b — — Merged a single-line update to exclude the .direnv directory from codespell checks. Approved by one reviewer and passed CI.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Added codespell exclusions for the .direnv directory to static analysis checks. The change was integrated directly without comments or revisions.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#2985 (confidence 99%)

    Excluded .direnv artifacts from Black and flake8 static checks; PR #3028 extended this treatment to codespell.

  • Caused By: snapcraft#2980 (confidence 95%)

    Added direnv-generated files to .gitignore before the related static-check exclusions were introduced.

Related issues

Issue Project State Summary Similarity
#35 lint: skip more commonly ignored files and directories with codespell starbase merged Merged PR #35 via commit ee3c053, expanding codespell's exclusions with commonly ignored files and directories from Snapcraft.
72%
#2980 vcs: add direnv files to .gitignore snapcraft merged Merged PR #2980 (commit d652ca825) added .direnv to snapcraft's .gitignore so direnv-generated environment artifacts stay out of version control; follow-ups #2985 and #3028 excluded .direnv from static/codespell checks.
71%