-
Notifications
You must be signed in to change notification settings - Fork 653
Conversation
✅ Deploy Preview for docs-rometools ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
523db10
to
cf64e21
Compare
Co-authored-by: l3ops <leops@users.noreply.github.com>
Co-authored-by: Micha Reiser <micha@rome.tools>
Co-authored-by: Micha Reiser <micha@rome.tools>
Co-authored-by: Micha Reiser <micha@rome.tools>
Co-authored-by: Micha Reiser <micha@rome.tools> Co-authored-by: l3ops <leops@users.noreply.github.com>
website/src/pages/configuration.mdx
Outdated
|
||
It configures where the formatter prints semicolons: | ||
- `always`, the semicolons is always added at the end of each statement; | ||
- `as-needed`, the semicolons are added only in places where it's needed, to protect from [ASI](https://en.wikibooks.org/wiki/JavaScript/Automatic_semicolon_insertion) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The value is asNeeded
(only the cli option is as-needed
).
@leops can you extend the editor README with an entry for sorting imports on save? "editor.codeActionsOnSave": {
"source.organizeImports.rome": true
} |
As part of this I tried to use the |
Co-authored-by: Micha Reiser <micha@rome.tools>
47ae4a1
to
140ea48
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one last typo and it should be good to merge
Summary
Closes #3716
@rome/staff please add/change the sections you want
Test Plan
Rendered changelog
Website