← Back to issue list

fix: remove trailing spaces where supported

View original Github issue

Metadata

Project
craft-cli
Number
#210
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged to remove trailing spaces where supported. Approved by two reviewers and passed CI checks. Modified three files. A follow-up PR was created to address cursor jumping behavior noted during review.

Suggested action: —

No scores available.

Issue body

Fixes #136 CRAFT-3358 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`?

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged to remove trailing spaces where supported. Approved by two reviewers and passed CI checks. Modified three files. A follow-up PR was created to address cursor jumping behavior noted during review.
qwen3.6-35b-a3b-mtp-q6 — — Merged to remove trailing spaces where supported, resolving issues #136 and CRAFT-3358. Test coverage remains at 94.83%. A follow-up PR #275 addresses the cursor jumping behavior observed during review.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#274 fix: remove dash in tests starbase merged Merged a hotfix removing a rogue dash from test files to prevent CI workflow failures. Approved by two reviewers, passed all checks, and modified one file.
72%
#2222 style: fix linting issues charmcraft merged Merged following approval by two reviewers. Resolved linting issues across three files with minor style adjustments (+8/-9 lines). No review comments or CI checks were required.
71%
#53 Fix typos craft-archives merged Merged after approval and passing CI checks. The PR corrected four typos across one file. A brief discussion regarding a missing commit from a rebase was clarified as a duplicate in another branch, with no impact on the merge.
70%
#117 a couple of fixes for tests charmcraft merged Merged to resolve issue #105. Approved by two reviewers, the pull request implements minor test corrections across two files with a +10/-1 line change.
70%