As of Drupal 8 all JS needs to follow our JavaScript coding standards. For this we use ESLint to check most of our standards.
Attached is the full list of errors found by ESLint and the patch that fixes them.

CommentFileSizeAuthor
igrowl.patch417 bytesNikhilesh Gupta
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Nikhilesh Gupta created an issue. See original summary.

kevinquillen’s picture

kevinquillen’s picture

Status: Needs review » Fixed

Thanks for that. I've ran ESLint and applied additional fixes.

Where is the doc page on drupal.org where you found the walkthrough for ESLint?

Nikhilesh Gupta’s picture

You can know regarding ESLint here ESLint

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

kevinquillen’s picture