-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add ESLint support and configure it to use Drupal standards #318
Comments
hkirsman
added a commit
that referenced
this issue
Oct 28, 2022
I've created PR and waiting for reviews @tormi Who could I invite? |
hkirsman
added a commit
that referenced
this issue
Oct 31, 2022
tormi
added a commit
that referenced
this issue
Nov 1, 2022
tormi
added a commit
that referenced
this issue
Nov 1, 2022
hkirsman
added a commit
that referenced
this issue
Nov 4, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've started to work on adding javascript linter that uses rules from Drupal here wunderio/code-quality#84
I think we don't have any verification for javascript files out of the box so we should start now. Better late than never.
Drupal project is good case to test and have a discussion.
So an example let's say we have this js file:
var foo="bar"
With ESLint and Drupal rules from web/core/.eslintrc.passing.json, this would be alerted and prompted to be fixed (automatically) to this:
const foo = 'bar'
This is just one small example.
The text was updated successfully, but these errors were encountered: