tools: update eslint

.eslintrc.js

@@ -114,6 +114,7 @@ module.exports ={
  tabWidth: 2,
  }],
  'new-parens': 'error',
+ 'no-async-promise-executor': 'error',
  'no-class-assign': 'error',
  'no-confusing-arrow': 'error',
  'no-const-assign': 'error',
@@ -147,7 +148,7 @@ module.exports ={
  'no-octal': 'error',
  'no-path-concat': 'error',
  'no-proto': 'error',
- 'no-redeclare': 'error',
+ 'no-redeclare': ['error',{'builtinGlobals': false }],
  'no-restricted-modules': ['error', 'sys'],
  /* eslint-disable max-len */
  'no-restricted-properties': [
@@ -246,6 +247,7 @@ module.exports ={
  'no-return-await': 'error',
  'no-self-assign': 'error',
  'no-self-compare': 'error',
+ 'no-shadow-restricted-names': 'error',
  'no-tabs': 'error',
  'no-template-curly-in-string': 'error',
  'no-this-before-super': 'error',

doc/api/cli.md

@@ -218,22 +218,23 @@ or be removed in the future.
 
 To avoid these cases, any builtin function overrides should be defined upfront:
 
-<!-- eslint-disable no-redeclare -->
 ```js
 const o ={};
 // THROWS: Cannot assign read only property 'toString' of object
 o.toString = () => 'string'
 
-// OK
-const o ={toString: () => 'string'};
-
 class X{
  constructor(){
  this.toString = () => 'string'
  }
 }
 // THROWS: Cannot assign read only property 'toString' of object
 new X();
+```
+
+```js
+// OK
+const o ={toString: () => 'string'};
 
 class X{
  toString = undefined;

tools/lint-js.js

@@ -3,7 +3,7 @@
 const rulesDirs = ['tools/eslint-rules'];
 const extensions = ['.js', '.md'];
 // This is the maximum number of files to be linted per worker at any given time
-const maxWorkload = 40;
+const maxWorkload = 60;
 
 const cluster = require('cluster');
 const path = require('path');
@@ -35,7 +35,8 @@ if (cluster.isMaster){
  let curPath = 'Starting ...'
  let showProgress = true;
  const globOptions ={
- nodir: true
+ nodir: true,
+ ignore: '**/node_modules/**/*'
  };
  const workerConfig ={};
  let startTime;