← Back to issue list

lint: update and freeze codespell

View original Github issue

Metadata

Project
snapcraft
Number
#4059
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

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).

Resolution: closed resolved

Reason: The PR was approved and merged into main (commit 423d008 / 255255f7b "lint: update and freeze codespell"), bumping the pinned codespell version across 4 files; the identical change was backported to the 7.3 hotfix branch by PR #4063 (1914eb975), so the lint CI failure it targeted was resolved.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `pytest tests/unit`? ----- Based on https://github.com/snapcore/snapcraft/actions/runs/4367512308/jobs/7640641055

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved 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).
stealth/space-bunny-alpha — closed resolved Maintainer PR #4059 merged into main as commit 255255f7b, updating and freezing the codespell pre-commit hook to a fixed revision after failures in CI run 4367512308; backported to hotfix/7.3 via #4063 (1914eb975).
stealth/space-bunny-alpha — closed resolved Merged PR #4059 updated and pinned codespell (423d008); the same change was backported to hotfix/7.3 by #4063 (1914eb975).
qwen/qwen3.6-35b-a3b — — Merged pull request updating and freezing the codespell linting tool. Approved by one reviewer and passing CI, the change modified four files (+4/-4 lines) and was successfully integrated into the main branch.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. The pull request updates and freezes the codespell linting tool to standardize spell-checking. Codecov reported a 0.00% coverage increase upon merge.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#4063 (confidence 95%)

    Backport of the same codespell pin/upgrade to the hotfix/7.3 branch, commit 1914eb975.

  • Related To: canonical/snapcraft#4011 (confidence 55%)

    Nearby dependency update PR shortly before this one; same codespell pin area in requirements files.

  • Related To: canonical/snapcraft#3028 (confidence 45%)

    Prior codespell configuration work (adding static-check excludes), part of the same lint configuration surface.

Related issues

Issue Project State Summary Similarity
#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.
90%
#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%
#685 build(deps): update dependency codespell to v2.3.0 (hotfix/1.20) craft-providers merged Renovate PR #685 merged into hotfix/1.20, bumping codespell from 2.2.6 to 2.3.0 in the lint dev dependency (+1/-1 line, approved, no CI failures). Same bump already landed on main via #621.
77%
#389 build(deps): update codespell craft-application merged Codespell dependency update merged in #389, changing two files; resolving commit: 70018156.
75%
#1296 ci: enable codespell charmcraft merged Merged PR #1296, fixing spelling errors in 3f10087f and enabling codespell in CI in 1d39e13d.
75%
#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%
#1236 build(deps): update dependency codespell to v2.4.2 (hotfix/1.5) rockcraft closed Codespell 2.4.2 update #1236 was closed unmerged after 113 days; tests and linters were failing, and no resolving commit was found.
72%
#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.
72%
#616 codespell charmcraft merged Merged PR #616 (commit c5e26d50), correcting codespell-reported typos across 16 files (+37/-37 lines).
70%
#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.
70%