← Back to issue list

build(deps): remove yamllint

View original Github issue

Metadata

Project
craft-platforms
Number
#144
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #144 removed yamllint and updated the pre-commit configuration from Starbase in commit 35b75fc.

Resolution: closed resolved

Reason: The approved PR was merged with no failing or pending checks, resolving the dependency cleanup in commit 35b75fc.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Plus the latest pre-commit config from starbase - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #144 removed yamllint and updated the pre-commit configuration from Starbase in commit 35b75fc.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR removing yamllint dependency and updating pre-commit configuration to latest starbase config.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#11 tools: add yamllint starbase merged Merged PR canonical/starbase#11 as commit 5a0d09f, adding optional yamllint tooling and configuration across three files.
82%
#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.
81%
#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.
81%
#2355 build(deps): update pre-commit hook adrienverge/yamllint to v1.37.1 (main) charmcraft merged Merged PR #2355 in commit bd4f9586, updating the yamllint pre-commit hook from v1.35.1 to v1.37.1.
79%
#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.
78%
#312 build(deps): update pre-commit hook adrienverge/yamllint to v1.37.1 (main) craft-store merged Renovate PR #312 bumping the adrienverge/yamllint pre-commit hook from v1.35.1 to v1.37.1 was merged into main (commit 7e5f289), a one-line .pre-commit-config.yaml bump; all CI checks passed.
78%
#131 build(deps): update bugfixes craft-platforms merged Renovate PR #131 merged into main as commit 82e62f6, bumping pre-commit hooks: adrienverge/yamllint v1.37.0->v1.37.1 and astral-sh/ruff-pre-commit v0.11.8->v0.11.10 (2 files/2 lines).
78%
#354 build(deps): update pre-commit hook adrienverge/yamllint to v1.38.0 (main) craft-store merged Merged PR #354 in commit ac20fd5, updating the adrienverge/yamllint pre-commit hook from v1.37.1 to v1.38.0.
78%
#291 lint: add shellcheck, yamllint craft-providers merged Merged PR #291 in commit 68ac1cc, adding shellcheck and yamllint from Starbase to address #287.
77%
#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.
76%