← Back to issue list

chore: remove unused yamllint config

View original Github issue

Metadata

Project
snapcraft
Number
#5489
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #5489 (commit 6bc01220e), deleting the now-unused 29-line yamllint configuration.

Resolution: closed resolved

Reason: PR #5489 was approved and merged, landing as commit 6bc01220e and removing the obsolete configuration file.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `make test`? --- Removes the now-unused yamllint config

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #5489 (commit 6bc01220e), deleting the now-unused 29-line yamllint configuration.
stealth/space-bunny-alpha — closed resolved Merged canonical/snapcraft#5489 (commit 6bc01220e), deleting the unused 29-line yamllint configuration.
qwen/qwen3.6-35b-a3b — — Merged after approval by two reviewers. The update removes an unused yamllint configuration file, deleting 29 lines across one file. The change passed most CI checks and was successfully integrated into the repository.
qwen3.6-35b-a3b-mtp-q6 — — Merged maintainer bepri's cleanup commit removing the unused yamllint configuration file. The change was successfully integrated into the main branch without requiring tests or additional reviews.

Update history

No update history recorded yet.

Related work

  • Caused By: canonical/snapcraft#362 (confidence 95%)

    Commit 8e2c46963 previously removed the unused yamllint dependency, leaving this configuration obsolete.

Related issues

Issue Project State Summary Similarity
#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.
86%
#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.
81%
#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.
78%
#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.
77%
#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).
76%
#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.
75%
#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.
74%
#144 build(deps): remove yamllint craft-platforms merged Merged PR #144 removed yamllint and updated the pre-commit configuration from Starbase in commit 35b75fc.
74%
#6079 chore: delete unused file snapcraft merged Merged PR #6079 as commit 75686f3da, deleting a three-line file unused since Snapcraft moved to a pyproject-based setup.
72%
#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.
72%