← Back to issue list

ci: enable codespell

View original Github issue

Metadata

Project
charmcraft
Number
#1296
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1296, fixing spelling errors in 3f10087f and enabling codespell in CI in 1d39e13d.

Resolution: closed resolved

Reason: PR #1296 was approved and merged. Its two commits corrected existing spelling errors and added codespell enforcement to CI.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This is broken into two commits, first fixing spelling and then enabling codespell.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1296, fixing spelling errors in 3f10087f and enabling codespell in CI in 1d39e13d.
stealth/space-bunny-alpha — closed resolved Merged charmcraft#1296 after two approvals, enabling codespell in CI and fixing spelling errors across 14 files in two commits.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged two commits to enable codespell in CI. The first fixes existing typos, and the second activates the spell-checking tool in the continuous integration pipeline.

Update history

No update history recorded yet.

Related work

  • Caused By: charmcraft#1305 (confidence 95%)

    Merged follow-up #1305 fixed a processor misspelling exposed by the newly enabled codespell check.

Related issues

Issue Project State Summary Similarity
#1770 Add codespell support snapcraft merged GCI contribution merged as commit 795628227: fixed repo-wide spelling errors and added codespell, first run standalone in Travis via tools/travis/run_codespell.sh with -q 4 and debian/changelog excluded.
77%
#167 lint:fix codespell typos craft-providers merged Merged PR #167 (commit ab325f2) fixing codespell-reported typos in 2 files (+2/-2), approved by a maintainer reviewer.
77%
#616 codespell charmcraft merged Merged PR #616 (commit c5e26d50), correcting codespell-reported typos across 16 files (+37/-37 lines).
75%
#4059 lint: update and freeze codespell snapcraft merged PR #4059 (maintainer lengau) updated and pinned/froze the codespell lint version to fix CI spelling-check failures; merged into main as commit 423d008 (255255f7b) and backported to hotfix/7.3 by #4063 (1914eb975).
75%
#389 build(deps): update codespell craft-application merged Codespell dependency update merged in #389, changing two files; resolving commit: 70018156.
74%
#728 build(deps): update dependency codespell to v2.4.1 (main) craft-application merged Renovate PR merged, bumping codespell 2.3.0 -> 2.4.1 via commit 4aa9d0f2 (2 files, +955/-955); all CI checks passed. Later superseded by #1141 (codespell 2.4.3).
74%
#132 build(deps): update dependency codespell to v2.3.0 (main) craft-archives merged Merged PR #132 as commit d6c69db, updating codespell from 2.2.5 to 2.3.0.
73%
#4063 lint: update and freeze codespell (#4059) snapcraft merged Backport PR merged as commit 1914eb975 into hotfix/7.3, applying the codespell update/pin from upstream PR #4059 (commit 255255f7b). +3/-3 lines, approved, no coverage change.
72%
#2162 tests: new codespell, narrowed checks and better execution order snapcraft merged PR #2162 merged into master (44ae843f) by sergiusens: bumped codespell to 0.12.0, ran mypy before spell checking, narrowed checked directories, and fixed newly found spelling errors in runtests.sh/static checks.
72%
#389 style: change spelling for updated codespell rockcraft merged Merged PR #389 with a one-line spelling change from “reuse” to “re-use”, clearing the codespell failure after the codespell update in PR #388.
72%