github-advanced-security[bot] commented on code in PR #154:
URL: https://github.com/apache/cordova-browser/pull/154#discussion_r3064169705


##########
lib/create.js:
##########
@@ -21,71 +19,58 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
-const ROOT = path.join(__dirname, '..', '..');
-const events = require('cordova-common').events;
+const ROOT = path.join(__dirname, '..');
+const { CordovaError, events } = require('cordova-common');
 const check_reqs = require('./check_reqs');
 
 // exported method to create a project, returns a promise that resolves with 
null
-module.exports.createProject = function (project_path, package_name, 
project_name) {
-/*
-    // create the dest and the standard place for our api to live
-    // platforms/platformName/cordova/Api.js
-*/
+module.exports.createProject = function (project_path, package_name, 
project_name, opts, root_config) {
+    opts = opts || {};

Review Comment:
   ## CodeQL / Useless assignment to local variable
   
   The value assigned to opts here is unused.
   
   [Show more 
details](https://github.com/apache/cordova-browser/security/code-scanning/4)



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to