tools: use arrow function for callback in lint-sh.js

The function declaration inside an else block is odd (and violates a recommended ESLint rule). We tend to use arrow functions for callbacks anyway, so switch to that. PR-URL: nodejs#41256 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
Linkgoron · Jan 31, 2022 · 5aebe75 · 5aebe75
1 parent cbfea63
commit 5aebe75
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tools/lint-sh.js b/tools/lint-sh.js
@@ -166,11 +166,11 @@ if (
   const entryPoint = resolve(process.argv[2]);
   const stats = statSync(entryPoint, { throwIfNoEntry: false });
 
-  function onError(e) {
+  const onError = (e) => {
     console.log(USAGE_STR);
     console.error(e);
     process.exitCode = 1;
-  }
+  };
   if (stats?.isDirectory()) {
     SPAWN_OPTIONS.cwd = entryPoint;
     checkFiles(...findScriptFilesRecursively(entryPoint)).catch(onError);