Repository navigation
[formatting] Raise default width to 100 columns - #637
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-01a11720-fd70-73ae-a723-70ad4084ac5b Co-authored-by: Amp <amp@ampcode.com>
Granite ReviewCouncil
FindingsNo actionable findings. |
Compatibility regression reportPackage set ✅ The candidate introduces no compatibility errors.
Introduced errorsNone. Fixed errors (0)None. Warning changes (0 introduced, 0 fixed)Introduced None. Fixed None. Candidate errors (0)None. Candidate warnings (36)
|
|
Review against AGENTS.md: no issues found. The default lives in Checks run:
|
Raises the formatter's default line width from 80 to 100 columns.
Why: 80 columns is narrower than modern editors and terminals comfortably support, so default-formatted output wraps more aggressively than most users want. 100 keeps lines readable while reducing forced breaks.
Approach: changes the single default in
formatting::Config::default, which the CLI (--widthfallback) and the language server (lineWidthfallback) both inherit. Help text, the formatting guide, and the two affected snapshots are updated; all other fixtures either pin an explicit width or already fit, so output is unchanged for them.Verification:
cargo check -p formatting --tests,-p iris-cli,-p iris-lsp-workspace;just t formatting,just t docs,just t lsp(no pending snapshots); E2Eformatting+lspsuites (25 tests) and the CLI help snapshot test pass;just formatandgit diff --checkclean.Thread: https://ampcode.com/threads/T-01a11720-fd70-73ae-a723-70ad4084ac5b