Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Document javascript.formatter.objectWrap and json.formatter.bracketSpacing #1744

Merged
merged 3 commits into from
Feb 16, 2025

Conversation

siketyan
Copy link
Member

@siketyan siketyan commented Feb 16, 2025

Summary

Part of #1737
Closes #1755
Closes #1754

This pull request adds documentation for the new formatter options I added in biomejs/biome#5068 and biomejs/biome#5083. Since biomejs/biome#5082 is not merged yet, not adding global formatter.objectWrap configuration for now.

Copy link

netlify bot commented Feb 16, 2025

Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit 2c855e8
🔍 Latest deploy log https://app.netlify.com/sites/biomejs/deploys/67b180f484265c00089b4a8f
😎 Deploy Preview https://deploy-preview-1744--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@siketyan siketyan force-pushed the feat/formatter-new-options branch from da6e443 to 57d28fb Compare February 16, 2025 05:48
@@ -425,6 +425,12 @@ The attribute position style in HTMLish languages.

> Default: `"auto"`

### `formatter.bracketSpacing`
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This option is already available in the current release of Biome, but was not documented yet.

@siketyan siketyan requested a review from a team as a code owner February 16, 2025 06:08
@ematipico ematipico merged commit a08a28c into biomejs:next Feb 16, 2025
7 checks passed
@siketyan siketyan deleted the feat/formatter-new-options branch February 16, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants