← Back to issue list

style: fix yamllint issues on 2.7 branch

View original Github issue

Metadata

Project
charmcraft
Number
#2424
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged after two approvals. Resolved yamllint compliance issues across 16 files on the 2.7 branch, applying 54 additions and 88 deletions. Core CI checks passed, integrating the style fix.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after two approvals. Resolved yamllint compliance issues across 16 files on the 2.7 branch, applying 54 additions and 88 deletions. Core CI checks passed, integrating the style fix.
qwen3.6-35b-a3b-mtp-q6 Fixed yamllint issues on the 2.7 branch and merged the changes.
qwen3.6-35b-a3b-mtp-q6 Merged a maintainer's pull request that fixed yamllint style violations on the 2.7 branch. The changes were accepted and integrated without additional review comments.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#11 tools: add yamllint starbase merged Merged maintainer lengau's addition of yamllint as an optional linting tool. Approved by two reviewers with zero unresolved comments. The change updates three files, adding 17 lines and removing 2, with no CI checks reported.
76%
#794 style: replace yamllint with prettier rockcraft merged Merged changes replacing yamllint with prettier for YAML formatting. Automated formatting updated 95 files, approved by three reviewers, and resolves #791. Test expectations were automatically regenerated to match prettier output.
76%
#2265 chore: migrate from yamllint to prettier charmcraft merged Merged after approval by two reviewers. Migrated YAML linting from yamllint to prettier across 22 files, standardizing formatting and reducing configuration overhead. Core CI checks passed.
75%
#1421 build: enable yamllint charmcraft merged Merged changes to enable yamllint in the build configuration. Approved by two reviewers, the update modifies 15 files (+41/-50 lines) to enforce YAML linting standards across the project.
75%
#20 style: fix linting issues craft-artifacts merged Merged after passing all CI checks and receiving approval from two reviewers. The change resolves linting issues across four files with a +11/-5 line diff, updating code style compliance.
74%
#305 style: replace yamllint with prettier starbase merged Merged after three approvals and passing CI. Replaces yamllint with prettier to auto-format YAML, JSON, JSON5, Markdown, and CSS across 13 files. Resolves #273. TOML support remains pending upstream fixes.
73%
#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.
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.
72%
#14 chore(ci): add shellcheck and yamllint snapcraft-rocks merged Merged. Resolves issue #9 by adding shellcheck and yamllint to the CI pipeline. Approved by one reviewer. Updates 5 files with 49 additions and 7 deletions.
70%