Skip to content

Commit

Permalink
Remove use of deprecated npm flag
Browse files Browse the repository at this point in the history
--omit=dev was introduced in npm 7 in 2021. This will break npm 6, however this is very EOL now, so don't think this is a major issue.

Signed-off-by: Chad Wilson <chadw@thoughtworks.com>
  • Loading branch information
chadlwilson committed Sep 23, 2023
1 parent 8148ae2 commit f5c2e7d
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions js.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,19 +19,19 @@
"windows": [
"npm",
"install",
"--production",
"--omit=dev",
"--silent"
],
"darwin": [
"npm",
"install",
"--production",
"--omit=dev",
"--silent"
],
"linux": [
"npm",
"install",
"--production",
"--omit=dev",
"--silent"
]
},
Expand All @@ -51,7 +51,7 @@
"--init"
]
},
"version": "3.0.1",
"version": "4.0.0",
"gaugeVersionSupport": {
"minimum": "1.0.7",
"maximum": ""
Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "gauge-js",
"version": "3.0.1",
"version": "4.0.0",
"description": "JavaScript runner for Gauge",
"main": "index.js",
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion scripts/install.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ var prepareFiles = function () {
try {
console.log("Installing dependencies...");
fs.removeSync("./node_modules");
child_process.execSync("npm install --production", { cwd: localPath() });
child_process.execSync("npm install --omit-dev", { cwd: localPath() });
} catch (err) {
console.error("Error installing dependencies: %s", err.toString());
console.error(err.stack);
Expand Down

0 comments on commit f5c2e7d

Please sign in to comment.