Skip to content

Commit 8c7e3eb

Browse files
author
Tim Etchells
committed
clean up some comments
1 parent 86ac49e commit 8c7e3eb

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

dev/src/extension.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import StringNamespaces from "./constants/strings/StringNamespaces";
2323
// your extension is activated the very first time the command is executed
2424
export async function activate(context: vscode.ExtensionContext): Promise<void> {
2525

26-
// TODO Any issue with having this in production?
2726
process.on("unhandledRejection", (err) => Log.e("Unhandled promise rejection:", err));
2827

2928
// Initialize our globals

dev/src/microclimate/connection/MCSocket.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -212,8 +212,7 @@ export default class MCSocket {
212212
{ projectName: project.name, startMode: StartModes.getUserFriendlyStartMode(startMode) }
213213
);
214214
Log.w(msg);
215-
// TODO show this warning or not?
216-
// vscode.window.showWarningMessage(msg);g
215+
// vscode.window.showWarningMessage(msg);
217216
}
218217
}
219218

dev/src/microclimate/project/DebugUtils.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,8 @@ export default class DebugUtils {
238238
address: project.connection.host,
239239
port: project.debugPort,
240240
localRoot: project.localPath.fsPath,
241-
// TODO user could change this in their dockerfile
241+
// User could change this in their dockerfile - but that would not fit with our import instructions
242+
// https://microclimate-dev2ops.github.io/importedprojects#nodejs-projects
242243
remoteRoot: "/app", // non-nls
243244
restart: true
244245
};

dev/src/microclimate/project/Project.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ export default class Project implements ITreeItemAdaptable, vscode.QuickPickItem
7070
this.name = projectInfo.name;
7171
this.id = projectInfo.projectID;
7272

73-
// TODO should use projectType not buildType but it's missing sometimes
73+
// should use projectType not buildType but it's missing sometimes
74+
// https://github.ibm.com/dev-ex/portal/issues/520
7475
this.type = new ProjectType(projectInfo.buildType, projectInfo.language);
7576

7677
this.localPath = vscode.Uri.file(

0 commit comments

Comments
 (0)