Skip to content

Commit d69244b

Browse files
committed
chore: fix comments
1 parent a521f36 commit d69244b

File tree

3 files changed

+20
-20
lines changed

3 files changed

+20
-20
lines changed

lib/common/declarations.d.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -1270,9 +1270,9 @@ interface IDashedOption {
12701270
*/
12711271
type: string;
12721272
/**
1273-
* Should skip in tracking
1273+
* Option has sensitive value
12741274
*/
1275-
private?: boolean;
1275+
hasSensitiveValue?: boolean;
12761276
/**
12771277
* Shorthand option passed on the command line with `-` sign, for example `-v`
12781278
*/

lib/helpers/options-track-helper.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ export class OptionsTracker {
3434
if (this.shouldSkipProperty(key, value, shorthands, optionsDefinitions)) {
3535
delete data[key];
3636
} else {
37-
if (options && optionsDefinitions[key] && optionsDefinitions[key].private) {
37+
if (options && optionsDefinitions[key] && optionsDefinitions[key].hasSensitiveValue) {
3838
value = OptionsTracker.PRIVATE_REPLACE_VALUE;
3939
} else if (key.toLowerCase().indexOf(OptionsTracker.PASSWORD_DETECTION_STRING) >= 0) {
4040
value = OptionsTracker.PRIVATE_REPLACE_VALUE;

lib/options.ts

+17-17
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@ export class Options {
1212
verbose: { type: OptionType.Boolean, alias: "v" },
1313
version: { type: OptionType.Boolean },
1414
help: { type: OptionType.Boolean, alias: "h" },
15-
profileDir: { type: OptionType.String, private: true },
15+
profileDir: { type: OptionType.String, hasSensitiveValue: true },
1616
analyticsClient: { type: OptionType.String },
17-
path: { type: OptionType.String, alias: "p", private: true },
17+
path: { type: OptionType.String, alias: "p", hasSensitiveValue: true },
1818
// This will parse all non-hyphenated values as strings.
1919
_: { type: OptionType.String }
2020
};
@@ -47,14 +47,14 @@ export class Options {
4747
framework: { type: OptionType.String },
4848
frameworkVersion: { type: OptionType.String },
4949
forDevice: { type: OptionType.Boolean },
50-
provision: { type: OptionType.Object, private: true },
50+
provision: { type: OptionType.Object, hasSensitiveValue: true },
5151
client: { type: OptionType.Boolean, default: true },
5252
env: { type: OptionType.Object },
5353
production: { type: OptionType.Boolean },
5454
debugTransport: { type: OptionType.Boolean },
5555
keyStorePath: { type: OptionType.String },
5656
keyStorePassword: { type: OptionType.String, },
57-
keyStoreAlias: { type: OptionType.String, private: true },
57+
keyStoreAlias: { type: OptionType.String, hasSensitiveValue: true },
5858
keyStoreAliasPassword: { type: OptionType.String },
5959
ignoreScripts: { type: OptionType.Boolean },
6060
disableNpmInstall: { type: OptionType.Boolean },
@@ -75,48 +75,48 @@ export class Options {
7575
androidTypings: { type: OptionType.Boolean },
7676
bundle: { type: OptionType.String },
7777
all: { type: OptionType.Boolean },
78-
teamId: { type: OptionType.Object, private: true },
78+
teamId: { type: OptionType.Object, hasSensitiveValue: true },
7979
syncAllFiles: { type: OptionType.Boolean, default: false },
8080
chrome: { type: OptionType.Boolean },
8181
inspector: { type: OptionType.Boolean },
8282
clean: { type: OptionType.Boolean },
8383
watch: { type: OptionType.Boolean, default: true },
8484
background: { type: OptionType.String },
85-
username: { type: OptionType.String, private: true },
85+
username: { type: OptionType.String, hasSensitiveValue: true },
8686
pluginName: { type: OptionType.String },
8787
hmr: { type: OptionType.Boolean },
8888
collection: { type: OptionType.String, alias: "c" },
8989
json: { type: OptionType.Boolean },
90-
avd: { type: OptionType.String, private: true },
90+
avd: { type: OptionType.String, hasSensitiveValue: true },
9191
// check not used
9292
config: { type: OptionType.Array },
9393
insecure: { type: OptionType.Boolean, alias: "k" },
9494
debug: { type: OptionType.Boolean, alias: "d" },
9595
timeout: { type: OptionType.String },
96-
device: { type: OptionType.String, private: true },
97-
availableDevices: { type: OptionType.Boolean, private: true },
98-
appid: { type: OptionType.String, private: true },
99-
geny: { type: OptionType.String, private: true },
96+
device: { type: OptionType.String, hasSensitiveValue: true },
97+
availableDevices: { type: OptionType.Boolean, hasSensitiveValue: true },
98+
appid: { type: OptionType.String, hasSensitiveValue: true },
99+
geny: { type: OptionType.String, hasSensitiveValue: true },
100100
debugBrk: { type: OptionType.Boolean },
101101
debugPort: { type: OptionType.Number },
102102
start: { type: OptionType.Boolean },
103103
stop: { type: OptionType.Boolean },
104-
ddi: { type: OptionType.String, private: true }, // the path to developer disk image
104+
ddi: { type: OptionType.String, hasSensitiveValue: true }, // the path to developer disk image
105105
justlaunch: { type: OptionType.Boolean },
106-
file: { type: OptionType.String, private: true },
106+
file: { type: OptionType.String, hasSensitiveValue: true },
107107
force: { type: OptionType.Boolean, alias: "f" },
108108
// remove legacy
109109
companion: { type: OptionType.Boolean },
110110
emulator: { type: OptionType.Boolean },
111111
sdk: { type: OptionType.String },
112-
template: { type: OptionType.String, private: true },
113-
certificate: { type: OptionType.String, private: true },
112+
template: { type: OptionType.String, hasSensitiveValue: true },
113+
certificate: { type: OptionType.String, hasSensitiveValue: true },
114114
certificatePassword: { type: OptionType.String },
115115
release: { type: OptionType.Boolean, alias: "r" },
116-
var: { type: OptionType.Object, private: true },
116+
var: { type: OptionType.Object, hasSensitiveValue: true },
117117
default: { type: OptionType.Boolean },
118118
count: { type: OptionType.Number },
119-
analyticsLogFile: { type: OptionType.String, private: true },
119+
analyticsLogFile: { type: OptionType.String, hasSensitiveValue: true },
120120
hooks: { type: OptionType.Boolean, default: true },
121121
link: { type: OptionType.Boolean, default: false },
122122
aab: { type: OptionType.Boolean }

0 commit comments

Comments
 (0)