style: add pre-commit to auto-formatting
Metadata
Current evaluation
Merged addition of pre-commit hooks for automated code formatting. Approved by two reviewers, passed all CI checks, and implemented with minimal changes across two files.
Suggested action: —
No scores available.
Issue body
- [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 |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged addition of pre-commit hooks for automated code formatting. Approved by two reviewers, passed all CI checks, and implemented with minimal changes across two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged pre-commit integration for automated code formatting. The maintainer verified linting and tests passed before finalizing the merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #68 tools: add pre-commit | starbase | merged | Merged addition of pre-commit configuration. Introduces three new files with forty-two lines of code, approved by one reviewer, and successfully integrated. | |
| #413 fix: walk back pre-commit all-files | starbase | merged | Merged a fix to remove --all-files from pre-commit formatting to align with make format scope. Approved by two reviewers, passed all CI checks, and merged with a single-line change. | |
| #441 build(deps): update pre-commit hooks | craft-application | merged | Merged update to pre-commit hooks. Approved by two reviewers with minimal changes (+4/-4 lines in one file). The maintainer-authored request passed checks and was successfully integrated. | |
| #20 style: fix linting issues | craft-artifacts | merged | Merged after passing all CI checks and receiving approval from two reviewers. The change resolves linting issues across four files with a +11/-5 line diff, updating code style compliance. | |
| #239 style(lint): autoformat | starbase | merged | Merged after approval. Applied automated linting to a single file, adjusting one line. The maintainer requested the merge post-review, completed without CI checks or unresolved comments. | |
| #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. | |
| #2222 style: fix linting issues | charmcraft | merged | Merged following approval by two reviewers. Resolved linting issues across three files with minor style adjustments (+8/-9 lines). No review comments or CI checks were required. | |
| #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. | |
| #197 build(deps): update pre-commit hook pre-commit/pre-commit-hooks to v6 | imagecraft | merged | Merged an automated dependency update for pre-commit-hooks from v5 to v6. Approved by two reviewers and passed most CI checks despite failing spread tests. |