← 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 a single-line update to exclude the .direnv directory from codespell checks. Approved by one reviewer and passed CI.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Merged. Added codespell exclusions for the .direnv directory to the static analysis configuration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#35 lint: skip more commonly ignored files and directories with codespell starbase merged Merged an update to the codespell configuration that skips additional commonly ignored files and directories. Approved by a reviewer, passed CI checks, and applied with a minimal one-file change.
80%
#1296 ci: enable codespell charmcraft merged Merged following approval by two reviewers. Enables the codespell CI check across 14 files. The change fixes existing spelling errors and updates configuration in two separate commits.
76%
#389 build(deps): update codespell craft-application merged Merged after approval by two reviewers and passing CI checks. Updates the codespell dependency across two files with minor line adjustments.
75%
#389 style: change spelling for updated codespell rockcraft merged Merged a one-line spelling correction to comply with updated codespell rules, resolving issue #388. Approved by three reviewers and passed CI checks.
72%
#1783 tests: run codespell as part of static tests snapcraft merged Merged following approval and passing CI. Integrates codespell into the static test suite to enforce spelling checks across five modified files.
72%
#2162 tests: new codespell, narrowed checks and better execution order snapcraft merged Updated codespell to 0.12.0, fixed spelling errors, narrowed search directories, and adjusted test execution order. Merged after approval and passing CI checks.
71%
#3941 lint: support the existence of 'venv' snapcraft merged Merged after reviewer approval and passing CI. Adds 'venv' and '.venv' to .gitignore and configures codespell to skip these common virtual environment directories.
70%