Skip to content

Commit 456031a

Browse files
author
Benjamin Pasero
committed
💄
1 parent 413e644 commit 456031a

7 files changed

Lines changed: 4 additions & 7 deletions

File tree

src/vs/base/common/marked/marked.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1242,7 +1242,7 @@ Parser.prototype.tok = function() {
12421242
return this.renderer.listitem(body);
12431243
}
12441244
case 'html': {
1245-
// TODO parse inline content if parameter markdown=1
1245+
// TODO@matt parse inline content if parameter markdown=1
12461246
return this.renderer.html(this.token.text);
12471247
}
12481248
case 'paragraph': {

src/vs/editor/common/model/pieceTreeTextBuffer/pieceTreeBase.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1100,7 +1100,7 @@ export class PieceTreeBase {
11001100
let endColumn = endOffset - this._buffers[0].lineStarts[endIndex];
11011101
let endPos = { line: endIndex, column: endColumn };
11021102
let newPiece = new Piece(
1103-
0, /** todo */
1103+
0, /** todo@peng */
11041104
start,
11051105
endPos,
11061106
this.getLineFeedCnt(0, start, endPos),

src/vs/platform/history/electron-main/historyMainService.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ export class HistoryMainService implements IHistoryMainService {
262262
}
263263
}
264264
} else if (Array.isArray(storedRecents.workspaces)) {
265-
// TODO legacy support can be removed at some point (6 month?)
265+
// TODO@martin legacy support can be removed at some point (6 month?)
266266
// format of 1.25 and before
267267
for (const workspace of storedRecents.workspaces) {
268268
if (typeof workspace === 'string') {

src/vs/workbench/parts/comments/electron-browser/commentsEditorContribution.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ export class ReviewController implements IEditorContribution {
326326
public onModelChanged(): void {
327327
this.localToDispose = dispose(this.localToDispose);
328328
if (this._newCommentWidget) {
329-
// todo store view state.
329+
// todo@peng store view state.
330330
this._newCommentWidget.dispose();
331331
this._newCommentWidget = null;
332332
}

src/vs/workbench/parts/feedback/electron-browser/media/feedback.css

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,6 @@
4848
padding-left: 3px;
4949
}
5050

51-
/* TODO @C5 review link color */
5251
.monaco-shell .feedback-form .content .channels a {
5352
padding: 2px 0;
5453
}

src/vs/workbench/parts/files/electron-browser/fileCommands.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@ export const ResourceSelectedForCompareContext = new RawContextKey<boolean>('res
7979
export const REMOVE_ROOT_FOLDER_COMMAND_ID = 'removeRootFolder';
8080
export const REMOVE_ROOT_FOLDER_LABEL = nls.localize('removeFolderFromWorkspace', "Remove Folder from Workspace");
8181

82-
//TODO #54483 support string paths for backward compatibility. check with @bpasero and remove if not necessary
8382
export const openWindowCommand = (accessor: ServicesAccessor, paths: (string | URI)[], forceNewWindow: boolean) => {
8483
const windowService = accessor.get(IWindowService);
8584
windowService.openWindow(paths.map(p => typeof p === 'string' ? URI.file(p) : p), { forceNewWindow });

src/vs/workbench/services/textmodelResolver/common/textModelResolverService.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ class ResourceModelCollection extends ReferenceCollection<TPromise<ITextEditorMo
8686

8787
return first(factories).then(model => {
8888
if (!model) {
89-
console.error(`Unable to open '${resource}' resource is not available.`); // TODO PII
9089
return TPromise.wrapError<ITextModel>(new Error('resource is not available'));
9190
}
9291

0 commit comments

Comments
 (0)