style: add pre-commit to auto-formatting
Metadata
Current evaluation
Merged PR #405 added pre-commit auto-formatting in commit 18e50d3, updating two files and passing all CI checks.
Resolution: closed resolved
Reason: PR #405 was approved and merged into the main branch as commit 18e50d3. Its pre-commit formatting configuration was implemented successfully, with all reported CI checks passing.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #405 added pre-commit auto-formatting in commit 18e50d3, updating two files and passing all CI checks. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #405 added pre-commit auto-formatting, updating two files with five additions and one deletion; all CI checks passed. | |
| 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 as commit 7a22e93, adding pre-commit tooling and configuration across three files. | |
| #411 style: autoformat with newest prettier | starbase | merged | Merged in canonical/starbase#411 as commit c860142, applying the newest Prettier formatting across three files; all CI checks passed. | |
| #239 style(lint): autoformat | starbase | merged | Merged PR #239 as commit 2439a13, applying a one-line autoformat change to one file after maintainer approval. | |
| #20 style: fix linting issues | craft-artifacts | merged | Merged PR #20 via commit 04cbdc0, applying style-only lint fixes across four files (+11/-5); both reviews approved and all CI checks passed. | |
| #413 fix: walk back pre-commit all-files | starbase | merged | Merged PR #413 in commit 2443e63, removing --all-files from pre-commit formatting so its scope matches make format. | |
| #292 ci: add pre-commit config | craft-providers | merged | Merged PR #292 as commit 638a1c9, adding Starbase's pre-commit configuration after #291 to advance the tooling sync tracked by #287. | |
| #531 Add shell formatting tool | starbase | closed | Added shfmt formatting and linting plus a pre-commit hook in merged PR #565 (commit e978069). | |
| #1204 style: fix linting issues | craft-parts | closed | Lint-only cleanup landed in commit a0622b31, matching PR #1204’s title; all reported CI checks passed. | |
| #2222 style: fix linting issues | charmcraft | merged | Merged PR #2222 as commit 31c109b4, applying style-only lint fixes across three files (+8/-9) with two approvals. | |
| #255 style: autoformat with ruff 0.9 | craft-store | merged | Merged PR #255 as commit 11c4c21, applying Ruff 0.9 autoformatting to five files (+22/-22) after two approvals. |