feat: document formatter.trailingNewline#3771
Conversation
✅ Deploy Preview for biomejs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
formatter.trailingNewline
WalkthroughThis change documents a new top-level formatter option `formatter.trailingNewline` and adds equivalent per-language options (`javascript.formatter.trailingNewline`, `json.formatter.trailingNewline`, `css.formatter.trailingNewline`, `graphql.formatter.trailingNewline`, `grit.formatter.trailingNewline`, `html.formatter.trailingNewline`) in the configuration reference. Each option describes adding a trailing newline at end-of-file, defaults to true, and includes a cautionary note against disabling it. Changes are documentation-only; no code or method signatures were modified. Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/content/docs/reference/configuration.mdx (1)
654-654: Use consistent default value format.The default value format "> Defaults to true." differs from the established convention used throughout this document. All other defaults use the format "> Default:
value" (see lines 51, 175, 182, 205, 242, etc.).Suggested format
Change all instances from:
> Defaults to true.To:
> Default: `true`This makes the format consistent with the rest of the document and properly formats the boolean value as code.
Also applies to: 896-896, 1105-1105, 1251-1251, 1328-1328, 1403-1403, 1569-1569
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/content/docs/reference/configuration.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: Redirect rules - biomejs
- GitHub Check: Header rules - biomejs
- GitHub Check: Pages changed - biomejs
- GitHub Check: test
Summary
Documentation for biomejs/biome#8334