feat(build): add shfmt tool for shell formatting
Metadata
Current evaluation
Merged #565 in commit e978069, adding shfmt formatting, a make format-shell target, and a pre-commit hook; this implemented #531.
Resolution: closed resolved
Reason: PR #565 was merged into main as commit e978069. It added shell formatting and linting support requested by #531, with all reported CI checks passing.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Implement shell formatting tool (shfmt) as requested in #531.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #565 in commit e978069, adding shfmt formatting, a make format-shell target, and a pre-commit hook; this implemented #531. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #565 added shfmt shell formatting, a make format-shell target, and a pre-commit hook, implementing #531 in commit e978069. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged implementation of the shfmt shell formatting tool per issue #531. Approved by one reviewer, passed all CI checks, and updated three files (+39/-4 lines). | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
45
Complexity:
25
Confidence:
65
|
needs review | Integrates shfmt shell formatting tool into the build system per issue #531. Currently pending maintainer review and CI validation with no comments or approvals. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/starbase#531
(confidence 100%)
PR #565 explicitly implemented the shell formatting tool request in #531.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #531 Add shell formatting tool | starbase | closed | Added shfmt formatting and linting plus a pre-commit hook in merged PR #565 (commit e978069). | |
| #1212 style: format shell scripts with shfmt | craft-application | merged | Merged PR applying shfmt formatting to repository shell scripts ahead of Starbase sync; +182/-182 across 4 files. | |
| #1022 style: reformat shell scripts with shfmt | craft-providers | merged | Merged PR #1022 to mechanically reformat shell scripts in craft-providers using shfmt v3.12.0. Changes are purely whitespace and quoting with no logic modifications. | |
| #505 test: make completion test robust to shfmt formatting | craft-cli | merged | Merged PR #505 fixes test_completion_output by formatting the expected fixture with shfmt and normalizing generated output, preventing byte-for-byte comparison failures after repo tooling reformatting. |