← Back to issue list

style: replace yamllint with prettier

View original Github issue

Metadata

Project
rockcraft
Number
#794
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged changes replacing yamllint with prettier for YAML formatting. Automated formatting updated 95 files, approved by three reviewers, and resolves #791. Test expectations were automatically regenerated to match prettier output.

Suggested action:

No scores available.

Issue body

This PR has a series of commits. The first one is a "manual cherry-pick" of [starbase/805](https://github.com/canonical/starbase/pull/305), and [this one](https://github.com/canonical/rockcraft/pull/794/commits/177b16121781ea6d3f5612a28ac482120328dacf) is the automated result of running `make format-prettier` (no manual changes). Spread failure is unrelated! Fixes #791 - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged changes replacing yamllint with prettier for YAML formatting. Automated formatting updated 95 files, approved by three reviewers, and resolves #791. Test expectations were automatically regenerated to match prettier output.
qwen3.6-35b-a3b-mtp-q6 Merged. Replaced yamllint with prettier for YAML formatting. Automated formatting updated templates and test fixtures, resolving issue #791 without manual intervention.
qwen3.6-35b-a3b-mtp-q6 Merged. Replaced yamllint with prettier for YAML formatting. Automated changes applied via make format-prettier, updating test expectations to match prettier output. Resolves issue #791.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#305 style: replace yamllint with prettier starbase merged Merged after three approvals and passing CI. Replaces yamllint with prettier to auto-format YAML, JSON, JSON5, Markdown, and CSS across 13 files. Resolves #273. TOML support remains pending upstream fixes.
90%
#2265 chore: migrate from yamllint to prettier charmcraft merged Merged after approval by two reviewers. Migrated YAML linting from yamllint to prettier across 22 files, standardizing formatting and reducing configuration overhead. Core CI checks passed.
86%
#791 Lint yaml files with prettier rockcraft closed Closed without implementation. The request to replace yamllint with prettier was superseded by an internal Jira ticket (CRAFT-3977) via automated sync. No code changes were merged.
79%
#411 style: autoformat with newest prettier starbase merged Merged after two approvals and passing CI. The pull request applied automatic formatting using the latest Prettier version, updating three files with a net reduction of 10 lines.
76%
#255 style: autoformat with ruff 0.9 craft-store merged Merged following two approvals. Applied ruff 0.9 autoformatting across five files, altering 22 lines. All CI checks passed. A maintainer noted that switching from yamllint to prettier will resolve remaining linting concerns.
76%
#2424 style: fix yamllint issues on 2.7 branch charmcraft merged Merged after two approvals. Resolved yamllint compliance issues across 16 files on the 2.7 branch, applying 54 additions and 88 deletions. Core CI checks passed, integrating the style fix.
76%
#1421 build: enable yamllint charmcraft merged 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.
75%
#11 tools: add yamllint starbase merged 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.
74%
#273 Replace `yamllint` with `prettier` starbase closed Closed without implementation. Scope expanded to include TOML, but no code changes, merge, or resolution details were recorded, indicating the request was abandoned.
72%