Skip to content

Commit 9d7c433

Browse files
alan-agius4clydin
alan-agius4
authored and
clydin
committed
docs: remove when true, when false from argument descriptions
It's confusing and redundant to use `When true` and `when false` in descriptions for a CLI arguments because specifying false/true is redundant in a command line argument flag and in most cases users will not do it. Example: `--foo=true` is the same as `--foo` `--foo=false` is the same as `--no-foo` (cherry picked from commit c99cef7)
1 parent 34cd0e0 commit 9d7c433

File tree

23 files changed

+86
-86
lines changed

23 files changed

+86
-86
lines changed

‎packages/angular/cli/commands/config.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
},
3232
"global": {
3333
"type": "boolean",
34-
"description": "When true, accesses the global configuration in the caller's home directory.",
34+
"description": "Access the global configuration in the caller's home directory.",
3535
"default": false,
3636
"aliases": ["g"]
3737
}

‎packages/angular/cli/commands/definitions.json‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
]
2222
},
2323
"prod": {
24-
"description": "Shorthand for \"--configuration=production\".\nWhen true, sets the build configuration to the production target.\nBy default, the production target is set up in the workspace configuration such that all builds make use of bundling, limited tree-shaking, and also limited dead code elimination.",
24+
"description": "Shorthand for \"--configuration=production\".\nSet the build configuration to the production target.\nBy default, the production target is set up in the workspace configuration such that all builds make use of bundling, limited tree-shaking, and also limited dead code elimination.",
2525
"type": "boolean"
2626
}
2727
}
@@ -42,13 +42,13 @@
4242
"type": "boolean",
4343
"default": false,
4444
"aliases": [ "d" ],
45-
"description": "When true, runs through and reports activity without writing out results."
45+
"description": "Run through and reports activity without writing out results."
4646
},
4747
"force": {
4848
"type": "boolean",
4949
"default": false,
5050
"aliases": [ "f" ],
51-
"description": "When true, forces overwriting of existing files."
51+
"description": "Force overwriting of existing files."
5252
}
5353
}
5454
},
@@ -57,12 +57,12 @@
5757
"interactive": {
5858
"type": "boolean",
5959
"default": "true",
60-
"description": "When false, disables interactive input prompts."
60+
"description": "Enable interactive input prompts."
6161
},
6262
"defaults": {
6363
"type": "boolean",
6464
"default": "false",
65-
"description": "When true, disables interactive input prompts for options with a default."
65+
"description": "Disable interactive input prompts for options with a default."
6666
}
6767
}
6868
}

‎packages/angular/cli/commands/doc.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
"aliases": ["s"],
2525
"type": "boolean",
2626
"default": false,
27-
"description": "When true, searches all of angular.io. Otherwise, searches only API reference documentation."
27+
"description": "Search all of angular.io. Otherwise, searches only API reference documentation."
2828
},
2929
"version" : {
3030
"oneOf": [

‎packages/angular/cli/commands/new.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
"type": "boolean",
2323
"default": false,
2424
"aliases": [ "v" ],
25-
"description": "When true, adds more details to output logging."
25+
"description": "Add more details to output logging."
2626
}
2727
},
2828
"required": []

‎packages/angular/cli/commands/update.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
"type": "boolean"
4545
},
4646
"migrateOnly": {
47-
"description": "Only perform a migration, does not update the installed version.",
47+
"description": "Only perform a migration, do not update the installed version.",
4848
"oneOf": [
4949
{
5050
"type": "boolean"

‎packages/angular/cli/lib/config/schema.json‎

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@
155155
},
156156
"skipTests": {
157157
"type": "boolean",
158-
"description": "When true, does not create test files.",
158+
"description": "Do not create test files.",
159159
"default": false
160160
}
161161
}
@@ -197,7 +197,7 @@
197197
},
198198
"skipTests": {
199199
"type": "boolean",
200-
"description": "When true, does not create test files.",
200+
"description": "Do not create test files.",
201201
"default": false
202202
}
203203
}
@@ -244,7 +244,7 @@
244244
},
245245
"skipTests": {
246246
"type": "boolean",
247-
"description": "When true, does not create test files.",
247+
"description": "Do not create test files.",
248248
"default": false
249249
}
250250
}
@@ -259,7 +259,7 @@
259259
},
260260
"skipTests": {
261261
"type": "boolean",
262-
"description": "When true, does not create test files.",
262+
"description": "Do not create test files.",
263263
"default": false
264264
},
265265
"skipImport": {
@@ -285,7 +285,7 @@
285285
"properties": {
286286
"skipTests": {
287287
"type": "boolean",
288-
"description": "When true, does not create test files.",
288+
"description": "Do not create test files.",
289289
"default": false
290290
}
291291
}
@@ -1210,13 +1210,13 @@
12101210
},
12111211
"open": {
12121212
"type": "boolean",
1213-
"description": "When true, open the live-reload URL in default browser.",
1213+
"description": "Open the live-reload URL in default browser.",
12141214
"default": false,
12151215
"alias": "o"
12161216
},
12171217
"liveReload": {
12181218
"type": "boolean",
1219-
"description": "When true, reload the page on change using live-reload.",
1219+
"description": "Reload the page on change using live-reload.",
12201220
"default": true
12211221
},
12221222
"publicHost": {
@@ -1237,27 +1237,27 @@
12371237
},
12381238
"disableHostCheck": {
12391239
"type": "boolean",
1240-
"description": "When true, don't verify that connected clients are part of allowed hosts.",
1240+
"description": "Do not verify that connected clients are part of allowed hosts.",
12411241
"default": false
12421242
},
12431243
"hmr": {
12441244
"type": "boolean",
1245-
"description": "When true, enable hot module replacement.",
1245+
"description": "Enable hot module replacement.",
12461246
"default": false
12471247
},
12481248
"watch": {
12491249
"type": "boolean",
1250-
"description": "When true, rebuild on change.",
1250+
"description": "Rebuild on change.",
12511251
"default": true
12521252
},
12531253
"hmrWarning": {
12541254
"type": "boolean",
1255-
"description": "When true, show a warning when the --hmr option is enabled.",
1255+
"description": "Show a warning when the --hmr option is enabled.",
12561256
"default": true
12571257
},
12581258
"servePathDefaultWarning": {
12591259
"type": "boolean",
1260-
"description": "When true, show a warning when deploy-url/base-href use unsupported serve path values.",
1260+
"description": "Show a warning when deploy-url/base-href use unsupported serve path values.",
12611261
"default": true
12621262
},
12631263
"optimization": {
@@ -1269,12 +1269,12 @@
12691269
"properties": {
12701270
"scripts": {
12711271
"type": "boolean",
1272-
"description": "When true, enable optimization of the scripts output.",
1272+
"description": "Enable optimization of the scripts output.",
12731273
"default": true
12741274
},
12751275
"styles": {
12761276
"type": "boolean",
1277-
"description": "When true, enable optimization of the styles output.",
1277+
"description": "Enable optimization of the styles output.",
12781278
"default": true
12791279
}
12801280
},
@@ -1290,25 +1290,25 @@
12901290
"description": "Build using ahead-of-time compilation."
12911291
},
12921292
"sourceMap": {
1293-
"description": "When true, output sourcemaps.",
1293+
"description": "Output sourcemaps.",
12941294
"default": true,
12951295
"oneOf": [
12961296
{
12971297
"type": "object",
12981298
"properties": {
12991299
"scripts": {
13001300
"type": "boolean",
1301-
"description": "When true, output sourcemaps for all scripts.",
1301+
"description": "Output sourcemaps for all scripts.",
13021302
"default": true
13031303
},
13041304
"styles": {
13051305
"type": "boolean",
1306-
"description": "When true, output sourcemaps for all styles.",
1306+
"description": "Output sourcemaps for all styles.",
13071307
"default": true
13081308
},
13091309
"vendor": {
13101310
"type": "boolean",
1311-
"description": "When true, resolve vendor packages sourcemaps.",
1311+
"description": "Resolve vendor packages sourcemaps.",
13121312
"default": false
13131313
}
13141314
},
@@ -1321,11 +1321,11 @@
13211321
},
13221322
"vendorChunk": {
13231323
"type": "boolean",
1324-
"description": "When true, use a separate bundle containing only vendor libraries."
1324+
"description": "Use a separate bundle containing only vendor libraries."
13251325
},
13261326
"commonChunk": {
13271327
"type": "boolean",
1328-
"description": "When true, use a separate bundle containing code used across multiple bundles."
1328+
"description": "Use a separate bundle containing code used across multiple bundles."
13291329
},
13301330
"baseHref": {
13311331
"type": "string",
@@ -1337,11 +1337,11 @@
13371337
},
13381338
"verbose": {
13391339
"type": "boolean",
1340-
"description": "When true, add more details to output logging."
1340+
"description": "Add more details to output logging."
13411341
},
13421342
"progress": {
13431343
"type": "boolean",
1344-
"description": "When true, log progress to the console while building."
1344+
"description": "Log progress to the console while building."
13451345
}
13461346
},
13471347
"additionalProperties": false

‎packages/schematics/angular/application/schema.json‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,13 @@
2020
"x-prompt": "What name would you like to use for the application?"
2121
},
2222
"inlineStyle": {
23-
"description": "When true, includes styles inline in the root component.ts file. Only CSS styles can be included inline. Default is false, meaning that an external styles file is created and referenced in the root component.ts file.",
23+
"description": "Include styles inline in the root component.ts file. Only CSS styles can be included inline. Default is false, meaning that an external styles file is created and referenced in the root component.ts file.",
2424
"type": "boolean",
2525
"alias": "s",
2626
"x-user-analytics": 9
2727
},
2828
"inlineTemplate": {
29-
"description": "When true, includes template inline in the root component.ts file. Default is false, meaning that an external template file is created and referenced in the root component.ts file. ",
29+
"description": "Include template inline in the root component.ts file. Default is false, meaning that an external template file is created and referenced in the root component.ts file. ",
3030
"type": "boolean",
3131
"alias": "t",
3232
"x-user-analytics": 10
@@ -39,7 +39,7 @@
3939
},
4040
"routing": {
4141
"type": "boolean",
42-
"description": "When true, creates a routing NgModule.",
42+
"description": "Create a routing NgModule.",
4343
"default": false,
4444
"x-prompt": "Would you like to add Angular routing?",
4545
"x-user-analytics": 17
@@ -76,7 +76,7 @@
7676
"x-user-analytics": 5
7777
},
7878
"skipTests": {
79-
"description": "When true, does not create \"spec.ts\" test files for the application.",
79+
"description": "Do not create \"spec.ts\" test files for the application.",
8080
"type": "boolean",
8181
"default": false,
8282
"alias": "S",
@@ -85,10 +85,10 @@
8585
"skipPackageJson": {
8686
"type": "boolean",
8787
"default": false,
88-
"description": "When true, does not add dependencies to the \"package.json\" file."
88+
"description": "Do not add dependencies to the \"package.json\" file."
8989
},
9090
"minimal": {
91-
"description": "When true, creates a bare-bones project without any testing frameworks. (Use for learning purposes only.)",
91+
"description": "Create a bare-bones project without any testing frameworks. (Use for learning purposes only.)",
9292
"type": "boolean",
9393
"default": false,
9494
"x-user-analytics": 14
@@ -100,7 +100,7 @@
100100
},
101101
"lintFix": {
102102
"type": "boolean",
103-
"description": "When true, applies lint fixes after generating the application.",
103+
"description": "Apply lint fixes after generating the application.",
104104
"x-user-analytics": 15,
105105
"x-deprecated": "Use \"ng lint --fix\" directly instead."
106106
},

‎packages/schematics/angular/class/schema.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
},
3030
"skipTests": {
3131
"type": "boolean",
32-
"description": "When true, does not create \"spec.ts\" test files for the new class.",
32+
"description": "Do not create \"spec.ts\" test files for the new class.",
3333
"default": false,
3434
"x-user-analytics": 12
3535
},
@@ -41,7 +41,7 @@
4141
"lintFix": {
4242
"type": "boolean",
4343
"default": false,
44-
"description": "When true, applies lint fixes after generating the class.",
44+
"description": "Apply lint fixes after generating the class.",
4545
"x-user-analytics": 15,
4646
"x-deprecated": "Use \"ng lint --fix\" directly instead."
4747
}

‎packages/schematics/angular/component/schema.json‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,14 @@
3434
"alias": "b"
3535
},
3636
"inlineStyle": {
37-
"description": "When true, includes styles inline in the component.ts file. Only CSS styles can be included inline. By default, an external styles file is created and referenced in the component.ts file.",
37+
"description": "Include styles inline in the component.ts file. Only CSS styles can be included inline. By default, an external styles file is created and referenced in the component.ts file.",
3838
"type": "boolean",
3939
"default": false,
4040
"alias": "s",
4141
"x-user-analytics": 9
4242
},
4343
"inlineTemplate": {
44-
"description": "When true, includes template inline in the component.ts file. By default, an external template file is created and referenced in the component.ts file.",
44+
"description": "Include template inline in the component.ts file. By default, an external template file is created and referenced in the component.ts file.",
4545
"type": "boolean",
4646
"default": false,
4747
"alias": "t",
@@ -95,18 +95,18 @@
9595
},
9696
"skipTests": {
9797
"type": "boolean",
98-
"description": "When true, does not create \"spec.ts\" test files for the new component.",
98+
"description": "Do not create \"spec.ts\" test files for the new component.",
9999
"default": false,
100100
"x-user-analytics": 12
101101
},
102102
"flat": {
103103
"type": "boolean",
104-
"description": "When true, creates the new files at the top level of the current project.",
104+
"description": "Create the new files at the top level of the current project.",
105105
"default": false
106106
},
107107
"skipImport": {
108108
"type": "boolean",
109-
"description": "When true, does not import this component into the owning NgModule.",
109+
"description": "Do not import this component into the owning NgModule.",
110110
"default": false,
111111
"x-user-analytics": 18
112112
},
@@ -128,18 +128,18 @@
128128
"export": {
129129
"type": "boolean",
130130
"default": false,
131-
"description": "When true, the declaring NgModule exports this component.",
131+
"description": "The declaring NgModule exports this component.",
132132
"x-user-analytics": 19
133133
},
134134
"entryComponent": {
135135
"type": "boolean",
136136
"default": false,
137-
"description": "When true, the new component is the entry component of the declaring NgModule.",
137+
"description": "The new component is the entry component of the declaring NgModule.",
138138
"x-deprecated": "Since version 9.0.0 with Ivy, entryComponents is no longer necessary."
139139
},
140140
"lintFix": {
141141
"type": "boolean",
142-
"description": "When true, applies lint fixes after generating the component.",
142+
"description": "Apply lint fixes after generating the component.",
143143
"x-user-analytics": 15,
144144
"x-deprecated": "Use \"ng lint --fix\" directly instead."
145145
}

0 commit comments

Comments
 (0)