Eslint just got a 2.0 version out: http://eslint.org/blog/2016/02/eslint-v2.0.0-released

The config has been changed a bit, we need to update our rules.

CommentFileSizeAuthor
core-js-eslintrc-update.patch1.18 KBnod_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nod_ created an issue. See original summary.

droplet’s picture

droplet CreditAttribution: droplet commented
Status: Needs review » Needs work

missing new `no-labels` ?

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Status: Needs work » Needs review

Already in the config

tic2000’s picture

tic2000 CreditAttribution: tic2000 at Intellix commented
Status: Needs review » Reviewed & tested by the community

I updated eslint, tried to eslint, lot of errors. I used the patch and the errors are gone (beside the annoying errors for empty space before anonymous function :P)

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi commented

RTBC+1, copied the updated config over to https://www.drupal.org/project/pareviewsh

  • catch committed 7512842 on 8.1.x 
    Issue #2667932 by nod_: Update eslint config for eslint 2.0 update

  • catch committed 73773d6 on 8.0.x 
    Issue #2667932 by nod_: Update eslint config for eslint 2.0 update

(...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.1.x and cherry-picked to 8.0.x, thanks!

Status: Fixed » Closed (fixed)

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