From b108f17d690f1b119a3e96e9898f75200817520c Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 14 Jun 2018 20:43:49 -0700 Subject: [PATCH] tools: do not disable `quotes` rule in .eslintrc.js Re-enable `quotes` rule in .eslintrc.js and fix code to abide by the rule. As a bonus, this makes the code (IMO, anyway) more readable. (It certainly isn't *less* readable, at least not IMO.) --- .eslintrc.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/.eslintrc.js b/.eslintrc.js index 9a4ffea210a828..dc83c0e8c11e90 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -147,32 +147,32 @@ module.exports = { message: '__defineSetter__ is deprecated.', } ], - /* eslint-disable max-len, quotes */ + /* eslint-disable max-len */ // If this list is modified, please copy the change to lib/.eslintrc.yaml 'no-restricted-syntax': [ 'error', { selector: "CallExpression[callee.object.name='assert'][callee.property.name='doesNotThrow']", - message: "Please replace `assert.doesNotThrow()` and add a comment next to the code instead." + message: 'Please replace `assert.doesNotThrow()` and add a comment next to the code instead.' }, { - selector: `CallExpression[callee.object.name='assert'][callee.property.name='rejects'][arguments.length<2]`, + selector: "CallExpression[callee.object.name='assert'][callee.property.name='rejects'][arguments.length<2]", message: 'assert.rejects() must be invoked with at least two arguments.', }, { - selector: `CallExpression[callee.object.name='assert'][callee.property.name='throws'][arguments.1.type='Literal']:not([arguments.1.regex])`, + selector: "CallExpression[callee.object.name='assert'][callee.property.name='throws'][arguments.1.type='Literal']:not([arguments.1.regex])", message: 'Use an object as second argument of assert.throws()', }, { - selector: `CallExpression[callee.object.name='assert'][callee.property.name='throws'][arguments.length<2]`, + selector: "CallExpression[callee.object.name='assert'][callee.property.name='throws'][arguments.length<2]", message: 'assert.throws() must be invoked with at least two arguments.', }, { - selector: `CallExpression[callee.name='setTimeout'][arguments.length<2]`, + selector: "CallExpression[callee.name='setTimeout'][arguments.length<2]", message: 'setTimeout() must be invoked with at least two arguments.', }, { - selector: `CallExpression[callee.name='setInterval'][arguments.length<2]`, + selector: "CallExpression[callee.name='setInterval'][arguments.length<2]", message: 'setInterval() must be invoked with at least 2 arguments.', }, {