-
Notifications
You must be signed in to change notification settings - Fork 653
Conversation
|
||
> Default: `80` | ||
|
||
#### `javascript.formatter.quoteStyle` |
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.
globals
have been ignored on purpose because we don't use it yet
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.
Given that this release is moving forward with state of the linter, should we add Linters
to the categories of the VSCode extension ?
Also if the goal of this PR is to kick off the release on merge, I think it needs to bump the version numbers of the LSP and CLI in order to trigger the corresponding workflows once the changes land on main
|
||
#### `linter.rules.js.recommended` | ||
|
||
Enables the recommended rules for the category `JavaScript`. |
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.
Is there a page we can link through so that people can see the recommended rules?
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.
Unfortunately not for the categories themselves
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.
I added a link to the list of rules
Co-authored-by: Micha Reiser <micha@rome.tools>
0fbaaf8
to
9af2c4a
Compare
Co-authored-by: Micha Reiser <micha@rome.tools>
Summary
Closes #2835
Test Plan