Skip to content

Commit 9ee3ee8

Browse files
chore: set correct TODOs in the code
1 parent 0b89ee5 commit 9ee3ee8

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

lib/common/commands/analytics.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ class AnalyticsCommand implements ICommand {
3131
switch (arg.toLowerCase()) {
3232
case "enable":
3333
await this.$analyticsService.setStatus(this.settingName, true);
34-
// await this.$analyticsService.track(this.settingName, "enabled");
34+
// TODO(Analytics): await this.$analyticsService.track(this.settingName, "enabled");
3535
this.$logger.info(`${this.humanReadableSettingName} is now enabled.`);
3636
break;
3737
case "disable":
38-
// await this.$analyticsService.track(this.settingName, "disabled");
38+
// TODO(Analytics): await this.$analyticsService.track(this.settingName, "disabled");
3939
await this.$analyticsService.setStatus(this.settingName, false);
4040
this.$logger.info(`${this.humanReadableSettingName} is now disabled.`);
4141
break;

lib/common/commands/proxy/proxy-base.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ export abstract class ProxyCommandBase implements ICommand {
1212

1313
protected async tryTrackUsage() {
1414
try {
15-
// TODO: Check why we have set the `disableAnalytics` to true and we track the command as separate one
15+
// TODO(Analytics): Check why we have set the `disableAnalytics` to true and we track the command as separate one
1616
// instead of tracking it through the commandsService.
1717
this.$logger.trace(this.commandName);
1818
// await this.$analyticsService.trackFeature(this.commandName);

lib/services/doctor-service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ class DoctorService implements IDoctorService {
3131
}
3232

3333
if (!configOptions || configOptions.trackResult) {
34-
// TODO: Consider sending this information to Google Analytics
34+
// TODO(Analytics): Consider sending this information to Google Analytics
3535
// await this.$analyticsService.track("DoctorEnvironmentSetup", hasWarnings ? "incorrect" : "correct");
3636
}
3737

0 commit comments

Comments
 (0)