Skip to content

Commit

Permalink
tools: specify rule disabled in test-assert.js
Browse files Browse the repository at this point in the history
Instead of disabling all ESLint rules on two lines in test-assert.js,
specify the rule that needs to be disabled.

PR-URL: nodejs#22563
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
Trott authored and addaleax committed Aug 31, 2018
1 parent c5bd856 commit f86d0eb
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test/parallel/test-assert.js
Original file line number Diff line number Diff line change
Expand Up @@ -692,7 +692,7 @@ common.expectsError(
() => {
a(
(() => 'string')()
// eslint-disable-next-line
// eslint-disable-next-line operator-linebreak
===
123 instanceof
Buffer
Expand All @@ -704,7 +704,7 @@ common.expectsError(
message: 'The expression evaluated to a falsy value:\n\n' +
' a(\n' +
' (() => \'string\')()\n' +
' // eslint-disable-next-line\n' +
' // eslint-disable-next-line operator-linebreak\n' +
' ===\n' +
' 123 instanceof\n' +
' Buffer\n' +
Expand All @@ -716,7 +716,7 @@ common.expectsError(
() => {
a(
(() => 'string')()
// eslint-disable-next-line
// eslint-disable-next-line operator-linebreak
===
123 instanceof
Buffer
Expand All @@ -728,7 +728,7 @@ common.expectsError(
message: 'The expression evaluated to a falsy value:\n\n' +
' a(\n' +
' (() => \'string\')()\n' +
' // eslint-disable-next-line\n' +
' // eslint-disable-next-line operator-linebreak\n' +
' ===\n' +
' 123 instanceof\n' +
' Buffer\n' +
Expand Down

0 comments on commit f86d0eb

Please sign in to comment.