build: enable yamllint
Metadata
Current evaluation
Merged PR #1421 in commit 4f9aa82b, enabling yamllint in Charmcraft's build checks and configuration.
Resolution: closed resolved
Reason: PR #1421 was approved by two reviewers and merged with no unresolved review comments. The change landed as commit 4f9aa82b.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1421 in commit 4f9aa82b, enabling yamllint in Charmcraft's build checks and configuration. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1421 (commit 4f9aa82b) by maintainer lengau, wiring yamllint into charmcraft's build/lint configuration across 15 files (+41/-50); approved by 2 reviewers with no unresolved comments. Later superseded by prettier migration. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged the build change enabling yamllint, with two reviewer approvals and no unresolved review comments. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged maintainer lengau's pull request to enable yamllint in the build configuration. The change was accepted and integrated without additional comments or revisions. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #144 build(deps): remove yamllint | craft-platforms | merged | Merged PR #144 removed yamllint and updated the pre-commit configuration from Starbase in commit 35b75fc. | |
| #11 tools: add yamllint | starbase | merged | Merged PR canonical/starbase#11 as commit 5a0d09f, adding optional yamllint tooling and configuration across three files. | |
| #122 build(deps): update pre-commit hook adrienverge/yamllint to v1.37.0 | craft-platforms | merged | Renovate PR #122 (merged as commit edef409) bumped the adrienverge/yamllint pre-commit hook from v1.35.1 to v1.37.0 in .pre-commit-config.yaml; yamllint was later removed entirely in #144. | |
| #868 build(deps): update dependency yamllint to v1.35.1 (main) | craft-parts | merged | Merged PR #868 upgraded yamllint from 1.32.0 to 1.35.1 in main via commit 60639226. | |
| #291 lint: add shellcheck, yamllint | craft-providers | merged | Merged PR #291 in commit 68ac1cc, adding shellcheck and yamllint from Starbase to address #287. | |
| #244 build(deps): update pre-commit hook adrienverge/yamllint to v1.35.1 | starbase | merged | Renovate bumped the yamllint pre-commit hook from v1.33.0 to v1.35.1; merged via commit 96d6635 (approved, +1/-1 in .pre-commit-config.yaml). Later yamllint was dropped entirely in #362. |