chore: remove unused yamllint dependency
Metadata
Current evaluation
Merged #362 in commit 8e2c469, removing the unused yamllint dependency from three files after YAML linting had moved to Prettier.
Resolution: closed resolved
Reason: PR #362 was merged as commit 8e2c469, deleting the unused yamllint dependency across three files. Earlier merged PR #305 had replaced yamllint with Prettier, making the cleanup possible.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #362 in commit 8e2c469, removing the unused yamllint dependency from three files after YAML linting had moved to Prettier. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/starbase#362 in commit 8e2c469, removing the unused yamllint dependency across three files. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a maintainer's change to remove the unused yamllint dependency. Approved by one reviewer, the update modified three files, removing 29 lines and adding 3. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the removal of the unused yamllint dependency. The change was integrated directly without comments or revisions. |
Update history
No update history recorded yet.
Related work
-
Caused By:
canonical/starbase#305
(confidence 98%)
PR #305 replaced yamllint with Prettier, leaving the dependency unused.
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. | |
| #2265 chore: migrate from yamllint to prettier | charmcraft | merged | Merged as commit 0e124cfd: pre-commit yamllint hook dropped and Prettier (3.6.0) added for yaml/yml/json/md formatting, ported from starbase; 22 files re-formatted. | |
| #178 chore(deps): update dependency yamllint to v1.33.0 | starbase | merged | Renovate bumped yamllint 1.31.0 -> 1.33.0; PR #178 merged to main in commit 0cfc30a, a one-line change to the yamllint pin in .pre-commit-config.yaml. | |
| #51 chore(deps): update dependency yamllint to v1.31.0 | starbase | merged | Renovate dependency bump yamllint 1.29.0 -> 1.31.0 merged into main as commit 9de5c85; one-line .pre-commit-config.yaml change, approved and no failing CI. Later superseded by #178 (v1.33.0). | |
| #375 chore(lint): Remove trailing whitespace | rockcraft | merged | Merged PR #375 removed trailing whitespace from one file (+4/-4), resolving yamllint complaints; landed as commit 777df7e3. | |
| #273 Replace `yamllint` with `prettier` | starbase | closed | Resolved by merged PR #305 (commit 68d8d79), replacing yamllint with Prettier for YAML, JSON/JSON5, Markdown and CSS; TOML was excluded due to prettier/prettier#15141. | |
| #546 chore(deps): update dependency yamllint to v1.32.0 | craft-parts | merged | Merged PR #546 updated yamllint from 1.29.0 to 1.32.0 via commit 115b32c5. | |
| #1144 chore(deps): update dependency yamllint to v1.32.0 | charmcraft | merged | Merged PR #1144, updating yamllint from 1.29.0 to 1.32.0 in commit 2c06b8c5. | |
| #77 chore(deps): update dependency lint/yamllint to v1.32.0 | starbase | merged | Renovate PR #77 merged, bumping lint/yamllint from 1.31.0 to 1.32.0 via commit 4477363. | |
| #614 chore(deps): update dependency yamllint to v1.33.0 | craft-parts | closed | yamllint 1.33.0 PR #614 was closed unmerged and ignored; #868 later upgraded the pin from 1.32.0 to 1.35.1 in commit 60639226. |