Skip to content

Commit b44f628

Browse files
committed
[INTERNAL] build: Add details to include-dependency options
As per feedback provided in SAP/ui5-tooling#841 (comment), explain that the provided dependency names must match the ui5.yaml names and not the npm module names.
1 parent 4727ef4 commit b44f628

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

lib/cli/commands/build.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,17 @@ build.builder = function(cli) {
2727
middlewares: [baseMiddleware]
2828
})
2929
.option("include-all-dependencies", {
30-
describe: "Include all dependencies in the build result",
30+
describe: "Include all dependencies in the build result. " +
31+
"This is equivalent to '--include-dependency \"*\"'",
3132
alias: ["all", "a"],
3233
default: false,
3334
type: "boolean"
3435
})
3536
.option("include-dependency", {
3637
describe: "A list of dependencies to be included in the build result. You can use the asterisk '*' as" +
3738
" an alias for including all dependencies in the build result. The listed dependencies cannot be" +
38-
" overruled by dependencies defined in 'exclude-dependency'.",
39+
" overruled by dependencies defined in 'exclude-dependency'. " +
40+
"The provided name must match with the dependency name shown in 'ui5 ls --flat'",
3941
type: "string",
4042
array: true
4143
})
@@ -54,7 +56,8 @@ build.builder = function(cli) {
5456
})
5557
.option("exclude-dependency", {
5658
describe: "A list of dependencies to be excluded from the build result. The listed dependencies can" +
57-
" be overruled by dependencies defined in 'include-dependency'.",
59+
" be overruled by dependencies defined in 'include-dependency'. " +
60+
"The provided name must match with the dependency name shown in 'ui5 ls --flat'",
5861
type: "string",
5962
array: true
6063
})

0 commit comments

Comments
 (0)