Follow-up to #2274195: Decide ESLint config

For some reason .eslintrc and .eslintignore are in the root directory, but they are only checking core files. To be consistent with phprs.xml, it should be moved to the core folder.

Files: 
CommentFileSizeAuthor
#2 i2575541-2.patch693 bytesattiks
PASSED: [[SimpleTest]]: [PHP 5.5 MySQL] 114,310 pass(es). View

Comments

attiks created an issue. See original summary.

attiks’s picture

Status: Active » Needs review
FileSize
693 bytes
PASSED: [[SimpleTest]]: [PHP 5.5 MySQL] 114,310 pass(es). View

Moved both files and updated the .eslintignore file.

nod_’s picture

I want module to be checked to being in root means contrib desn't have to duplicate the config all the time.

attiks’s picture

We need to find a way to run this on the core directory, excluding all contrib, so testbot can add it. Eventually testbot will start running for contrib as well, but they need to be able to alter the excluded dirs i think.

For the moment the .eslintignore contains very specific core directories, so it does not make sense to put it in the core directory.

nod_’s picture

oh for the testbot that's an easy fixeslint core/

the ignore file don't cascade like the config files so it's a bit tricky.

attiks’s picture

or we move the .eslintignore to the core folder and run eslint in that folder?

nod_’s picture

that works

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

anoopjohn’s picture

What was finally decided on this?

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.6 was released on February 1, 2017 and is the final full bugfix release for the Drupal 8.2.x series. Drupal 8.2.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.3.0 on April 5, 2017. (Drupal 8.3.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.3.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.