Skip to content

Commit

Permalink
benchmark: increase lint compliance
Browse files Browse the repository at this point in the history
Remove two eslint-disable comments by replacing string concatenation
with template literals. These changes are in catch blocks that are not
part of the actual code being benchmarked.

PR-URL: nodejs#23305
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
  • Loading branch information
Trott committed Oct 7, 2018
1 parent b155358 commit 1d96d7d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions benchmark/napi/function_args/index.js
Expand Up @@ -12,16 +12,14 @@ let napi;
try {
v8 = require('./build/Release/binding');
} catch (err) {
// eslint-disable-next-line no-path-concat
console.error(__filename + ': V8 Binding failed to load');
console.error(`${__filename}: V8 Binding failed to load`);
process.exit(0);
}

try {
napi = require('./build/Release/napi_binding');
} catch (err) {
// eslint-disable-next-line no-path-concat
console.error(__filename + ': NAPI-Binding failed to load');
console.error(`${__filename}: NAPI-Binding failed to load`);
process.exit(0);
}

Expand Down

0 comments on commit 1d96d7d

Please sign in to comment.