← Back to issue list

style: change spelling for updated codespell

View original Github issue

Metadata

Project
rockcraft
Number
#389
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

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.

Resolution: closed resolved

Reason: PR #389 was merged with three approvals and no unresolved review comments. It adopted the valid “re-use” spelling instead of changing codespell settings, resolving the lint failure linked to PR #388.

Evaluated by: mr-cal-bot

No scores available.

Issue body

"re-use" is a [perfectly valid spelling](https://en.wiktionary.org/wiki/re-use), but this is easier than changing codespell settings every time we disagree with it Fixes the codespell issue in #388 - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved 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.
stealth/space-bunny-alpha — closed resolved Merged PR #389 as commit 29f1d7d6, changing “re-use” to “reuse” in spread.yaml to satisfy the updated codespell check.
stealth/space-bunny-alpha — closed resolved Merged spelling update in canonical/rockcraft#389 (29f1d7d6), changing one line to satisfy updated codespell and resolve #388.
qwen/qwen3.6-35b-a3b — — Merged a one-line spelling correction to comply with updated codespell rules, resolving issue #388. Approved by three reviewers and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged a spelling update to align with new codespell rules, replacing re-use to prevent configuration overrides. Resolves codespell issue #388.

Update history

No update history recorded yet.

Related work

  • Caused By: canonical/rockcraft#388 (confidence 99%)

    Merged as 1a8bbf6b; its codespell patch update prompted the spelling adjustment required by the newer checker.

Related issues

Issue Project State Summary Similarity
#616 codespell charmcraft merged Merged PR #616 (commit c5e26d50), correcting codespell-reported typos across 16 files (+37/-37 lines).
75%
#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.
74%
#389 build(deps): update codespell craft-application merged Codespell dependency update merged in #389, changing two files; resolving commit: 70018156.
72%
#1296 ci: enable codespell charmcraft merged Merged PR #1296, fixing spelling errors in 3f10087f and enabling codespell in CI in 1d39e13d.
72%
#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).
70%
#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).
70%