← Back to issue list

chore(lint): Remove trailing whitespace

View original Github issue

Metadata

Project
rockcraft
Number
#375
Type
pull request
State
merged
Author
merkata
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #375 removed trailing whitespace from one file (+4/-4), resolving yamllint complaints; landed as commit 777df7e3.

Resolution: closed resolved

Reason: PR #375 was approved by two reviewers and merged into the project as commit 777df7e3.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Some lines have trailing white spaces and I've removed them, tools like yamllint complain. - [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 #375 removed trailing whitespace from one file (+4/-4), resolving yamllint complaints; landed as commit 777df7e3.
stealth/space-bunny-alpha — closed resolved Merged lint cleanup PR #375 (commit 777df7e3) removing trailing whitespace flagged by yamllint; +4/-4 lines in 1 file, approved by 2 reviewers with no unresolved comments.
stealth/space-bunny-alpha — closed resolved Merged lint cleanup removing trailing whitespace from one file (+4/-4), addressing yamllint complaints; approved by two reviewers.
qwen/qwen3.6-35b-a3b — — Merged changes removing trailing whitespace from one file to satisfy linters like yamllint. Approved by two reviewers, passed CI, and modified four lines.
qwen3.6-35b-a3b-mtp-q6 — — Merged an external contribution removing trailing whitespace to resolve yamllint complaints. The change was integrated into the codebase without additional review.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5489 chore: remove unused yamllint config snapcraft merged Merged PR #5489 (commit 6bc01220e), deleting the now-unused 29-line yamllint configuration.
81%
#585 chore(linter): misc fixes craft-parts merged Merged PR #585 with miscellaneous linter fixes across 125 files, preserving individual commits rather than squashing, as requested.
78%
#359 chore(lint): update linters and fix linting issues with new rules craft-parts merged Merged PR #359 in commit e7b93bde, updating linters and fixing violations introduced by the new rules across 13 files.
78%
#345 chore: hush some of the linter output starbase merged Merged PR #345 quieted noisy linter tasks so earlier lint steps no longer flooded terminal output; landed as commit c9e090a.
78%
#1776 chore: remove unused code charmcraft merged Merged PR #1776 as commit 027696be, deleting 17,571 lines of unused code across 36 files after maintainer review and approval.
77%
#362 chore: remove unused yamllint dependency starbase merged Merged #362 in commit 8e2c469, removing the unused yamllint dependency from three files after YAML linting had moved to Prettier.
77%
#1040 chore(merge): 2.4.3 into main craft-parts merged Merged version 2.4.3 into main via PR #1040, landing as commit fab7fa92 after two reviewer approvals.
76%
#369 chore: fix linting imagecraft merged Merged PR #369 removed two redundant casts (1 file, +2/-2), fixing Ty linting; the lint/files check passed.
76%
#1099 chore: fix linting craft-application merged Merged as 4c674c21, updating two files to clear Ty unused-directive lint errors; lint/files passed and review was approved.
76%
#117 chore: clean up cruft craft-actions closed Merged PR #117 as commit 37d00fc, removing 117 lines from nine files to clean repository cruft ahead of v1.
76%