fix: replace requireCordovaModule with require for Cordova 9

phonegap · Apr 16, 2019 · 4c41276 · 4c41276
1 parent acfed44
commit 4c41276
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/hooks/windows/check-arch.js b/hooks/windows/check-arch.js
@@ -2,8 +2,8 @@ module.exports = function(ctx) {
     if (ctx.opts && ctx.opts.platforms && ctx.opts.platforms.indexOf('windows') > -1
         && ctx.opts.options) {
         var path = require('path');
-        var shell = ctx.requireCordovaModule('shelljs');
-        var nopt = ctx.requireCordovaModule('nopt');
+        var shell = require('shelljs');
+        var nopt = require('nopt');
 
         // parse and validate args
         var args = nopt({
@@ -18,7 +18,7 @@ module.exports = function(ctx) {
             'buildConfig': String
         }, {}, ctx.opts.options.argv, 0);
 
-        // Check if --appx flag is passed so that we have a project build version override:  
+        // Check if --appx flag is passed so that we have a project build version override:
         var isWin10 = args.appx && args.appx.toLowerCase() === 'uap';
 
         // Else check "windows-target-version" preference:

diff --git a/package.json b/package.json
@@ -54,5 +54,9 @@
   "devDependencies": {
     "jasmine-node": "1.14.5",
     "pluginpub": "^0.0.9"
+  },
+  "dependencies": {
+    "nopt": "^4.0.1",
+    "shelljs": "^0.8.3"
   }
 }