Re: [PR] chore: bump @cordova/eslint-config [cordova-fetch]

["erisu (via GitHub)" <gi...@apache.org>]

erisu commented on code in PR #130:
URL: https://github.com/apache/cordova-fetch/pull/130#discussion_r2271924034


##########
index.js:
##########
@@ -133,8 +135,8 @@ async function resolvePathToPackage (name, basedir) {
 async function isNpmInstalled () {
     try {
         return await which('npm');
-    } catch (e) {
-        throw new CordovaError('"npm" command line tool is not installed: make sure it is accessible on your PATH.');
+    } catch (err) {
+        throw new CordovaError('"npm" command line tool is not installed: make sure it is accessible on your PATH.', err);

Review Comment:
   I re-reviewed this and it is correct.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org