tools: add yamllint
Metadata
Current evaluation
Merged PR canonical/starbase#11 as commit 5a0d09f, adding optional yamllint tooling and configuration across three files.
Resolution: closed resolved
Reason: The maintainer-authored PR received approval from two reviewers and merged into the main branch as commit 5a0d09f.
Evaluated by: mr-cal-bot
No scores available.
Issue body
I've been using yamllint myself, so I thought I'd offer this as an option
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR canonical/starbase#11 as commit 5a0d09f, adding optional yamllint tooling and configuration across three files. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged maintainer lengau’s optional yamllint tool addition in canonical/starbase#11, changing three files (+17/-2) with approval from two reviewers. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged yamllint as an optional YAML linting tool. The maintainer added it to standardize YAML validation workflows based on personal usage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #291 lint: add shellcheck, yamllint | craft-providers | merged | Merged PR #291 in commit 68ac1cc, adding shellcheck and yamllint from Starbase to address #287. | |
| #144 build(deps): remove yamllint | craft-platforms | merged | Merged PR #144 removed yamllint and updated the pre-commit configuration from Starbase in commit 35b75fc. | |
| #305 style: replace yamllint with prettier | starbase | merged | Merged PR canonical/starbase#305 (commit 68d8d79, CRAFT-3626) swapped yamllint for Prettier in pre-commit and common.mk, covering YAML, JSON, JSON5, CSS and Markdown; TOML excluded pending prettier/prettier#15141. Closes #273. | |
| #1421 build: enable yamllint | charmcraft | merged | Merged PR #1421 in commit 4f9aa82b, enabling yamllint in Charmcraft's build checks and configuration. | |
| #2424 style: fix yamllint issues on 2.7 branch | charmcraft | merged | Merged PR #2424 corrected yamllint issues across 16 files on the 2.7 branch. | |
| #286 ci: sync configs with starbase | craft-providers | merged | Merged PR #286 as commit 73c340c, syncing .gitignore, .editorconfig, and yamllint.yaml with Starbase as the first configuration synchronization. | |
| #794 style: replace yamllint with prettier | rockcraft | merged | Merged #794, replacing yamllint with Prettier and reformatting 95 files; it fixed #791. The change was ported from starbase#305 and generated with make format-prettier. | |
| #30 meta: update tooling from starbase | craft-application | merged | Merged Starbase tooling updates in canonical/craft-application#30 at commit 2f843183, including linting adjustments and coverage fixes. | |
| #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. | |
| #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. |