Skip to content
This repository was archived by the owner on Aug 7, 2021. It is now read-only.

Commit 5c7ebeb

Browse files
authored
fix(installer): show helper message for new dependencies (#122)
1 parent d28dba1 commit 5c7ebeb

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

Diff for: dependencyManager.js

+15
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
const helpers = require("./projectHelpers");
22

3+
const NEW_DEPS_MESSAGE = `
4+
A few development dependencies were added. \
5+
Install them before bundling your project.`;
6+
7+
const ALREADY_ADDED_MESSAGE = `\
8+
Some dependencies may have already been added. \
9+
If you want to force update them, please run "node_modules/.bin/update-ns-webpack".`;
10+
311
function forceUpdateProjectDeps(packageJson) {
412
return addProjectDeps(packageJson, true);
513
}
@@ -14,6 +22,8 @@ function addProjectDeps(packageJson, force = false) {
1422
deps = addDependency(deps, name, version, force);
1523
});
1624

25+
logHelperMessages();
26+
1727
return deps;
1828
}
1929

@@ -94,6 +104,11 @@ function getVersionWithoutPatch(version) {
94104
}
95105
}
96106

107+
function logHelperMessages(someAlreadyAdded) {
108+
console.info(NEW_DEPS_MESSAGE);
109+
console.info(ALREADY_ADDED_MESSAGE);
110+
}
111+
97112
module.exports = {
98113
forceUpdateProjectDeps,
99114
addProjectDeps,

0 commit comments

Comments
 (0)