Skip to content

Commit 48e75c3

Browse files
committed
Polish comments before merge
1 parent 57e9e98 commit 48e75c3

File tree

2 files changed

+0
-9
lines changed

2 files changed

+0
-9
lines changed

src/client/common/terminal/service.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -104,18 +104,15 @@ export class TerminalService implements ITerminalService, Disposable {
104104
return this.executeCommandInternal(commandLine);
105105
}
106106

107-
// Process Python code execution once REPL is detected.
108107
private startReplListener(): void {
109108
if (this.replDataListener) {
110109
return;
111110
}
112111

113112
let terminalData = '';
114-
115113
this.replDataListener = this.applicationShell.onDidWriteTerminalData((e) => {
116114
if (this.terminal && e.terminal === this.terminal) {
117115
terminalData += e.data;
118-
// Check for Python REPL prompt (>>>)
119116
if (/>>>\s*$/.test(terminalData)) {
120117
traceVerbose('Python REPL ready, detected >>> prompt');
121118
this.isReplReady = true;
@@ -232,7 +229,6 @@ export class TerminalService implements ITerminalService, Disposable {
232229
if (terminal === this.terminal) {
233230
this.terminalClosed.fire();
234231
this.terminal = undefined;
235-
// Reset REPL state when terminal closes
236232
this.isReplReady = false;
237233
this.disposeReplListener();
238234
this.pythonReplCommandQueue = [];

src/test/common/terminals/service.unit.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,6 @@ suite('Terminal Service', () => {
241241
terminalManager.setup((t) => t.createTerminal(TypeMoq.It.isAny())).returns(() => terminal.object);
242242

243243
await service.ensureTerminal();
244-
// Start executeCommand (sets up listener), then fire >>> prompt, then await
245244
const executePromise = service.executeCommand(textToSend, true);
246245
onDidWriteTerminalDataEmitter.fire({ terminal: terminal.object, data: '>>> ' });
247246
await executePromise;
@@ -265,7 +264,6 @@ suite('Terminal Service', () => {
265264
terminalManager.setup((t) => t.createTerminal(TypeMoq.It.isAny())).returns(() => terminal.object);
266265

267266
await service.ensureTerminal();
268-
// Start executeCommand (sets up listener), then fire >>> prompt, then await
269267
const executePromise = service.executeCommand(textToSend, true);
270268
onDidWriteTerminalDataEmitter.fire({ terminal: terminal.object, data: '>>> ' });
271269
await executePromise;
@@ -290,7 +288,6 @@ suite('Terminal Service', () => {
290288
terminalManager.setup((t) => t.createTerminal(TypeMoq.It.isAny())).returns(() => terminal.object);
291289

292290
await service.ensureTerminal();
293-
// Start executeCommand (sets up listener), then fire >>> prompt, then await
294291
const executePromise = service.executeCommand(textToSend, true);
295292
onDidWriteTerminalDataEmitter.fire({ terminal: terminal.object, data: '>>> ' });
296293
await executePromise;
@@ -324,7 +321,6 @@ suite('Terminal Service', () => {
324321
terminalManager.setup((t) => t.createTerminal(TypeMoq.It.isAny())).returns(() => terminal.object);
325322

326323
await service.ensureTerminal();
327-
// Start executeCommand (sets up listener), then fire >>> prompt, then await
328324
const executePromise = service.executeCommand(textToSend, true);
329325
onDidWriteTerminalDataEmitter.fire({ terminal: terminal.object, data: '>>> ' });
330326
await executePromise;
@@ -348,7 +344,6 @@ suite('Terminal Service', () => {
348344
terminalManager.setup((t) => t.createTerminal(TypeMoq.It.isAny())).returns(() => terminal.object);
349345

350346
await service.ensureTerminal();
351-
// Start executeCommand (sets up listener), then fire >>> prompt, then await
352347
const executePromise = service.executeCommand(textToSend, true);
353348
onDidWriteTerminalDataEmitter.fire({ terminal: terminal.object, data: '>>> ' });
354349
await executePromise;

0 commit comments

Comments
 (0)