← Back to issue list

lint: missing f-string prefix

View original Github issue

Metadata

Project
craft-parts
Number
#179
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged a one-line fix adding a missing f-string prefix to resolve a linting error. Approved by one reviewer, passed CI checks, and integrated into the codebase.

Suggested action: —

No scores available.

Issue body

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com> - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a one-line fix adding a missing f-string prefix to resolve a linting error. Approved by one reviewer, passed CI checks, and integrated into the codebase.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix for a missing f-string prefix lint error. The change, authored by maintainer mr-cal, was successfully integrated into the codebase following CLA verification.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#33 Linting imagecraft merged Merged a one-line linting fix from a maintainer. The change modified a single file with one addition and one deletion, requiring no reviews or CI checks before integration.
76%
#5 style(lint): fix linting issues starflow merged Merged after approval. Resolved linting issues by modifying one file with three additions and three deletions. No unresolved comments or CI checks were present.
75%
#242 style: fix ty linting error craft-platforms merged Merged a single-file change fixing a ty linting error. Approved by one reviewer and passed CI checks. The one-line modification resolves the style violation.
75%
#36 Lint imagecraft merged Merged a single-file linting adjustment with one addition and one deletion. The maintainer-submitted change required no reviews or CI checks before integration.
73%
#239 style(lint): autoformat starbase merged Merged after approval. Applied automated linting to a single file, adjusting one line. The maintainer requested the merge post-review, completed without CI checks or unresolved comments.
73%
#193 style: fix typo in comment craft-cli merged Merged a one-line fix for a comment typo. Approved by a reviewer, passed CI checks, and integrated into main without affecting code coverage.
73%
#77 lint: fix linting issues with up-to-date linters craft-store merged Merged a maintainer-submitted pull request resolving linting errors from updated linter versions. Approved by one reviewer, passed CI, and modified 11 files (+40/-24 lines).
72%
#4981 style(lint): fix ruff 0.6 linting errors snapcraft merged Merged after receiving two approvals and passing CI checks. Resolves ruff 0.6 linting errors with a minimal one-file change.
72%
#4453 fix(metadata): add f-string to debug message snapcraft merged Merged a fix adding a missing f-string to a debug metadata message. Passed linting, unit tests, and CI checks, and was approved by three reviewers before integration.
72%
#291 lint: remove leftover pylint directive craft-parts merged Merged a change removing a leftover pylint directive. The single-line deletion across one file received reviewer approval and was integrated after passing checks.
72%