microsoft / microsoft/vscode-python-environments

Leverage execution.exitCode for execution promise resolve

Offen
#169 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@anthonykim1 arbeitet bereits daran.

Seit 07.2.2025.

debt needs PR
Vorherrschende Sprache
TypeScript
Sterne
138
Forks
62
Ø Merge
1 T. 4 Std.
Gemergte PRs (30 T.)
35

Beschreibung

https://github.com/microsoft/vscode-python-environments/pull/160#discussion_r1946083357

We have
```
let timer: NodeJS.Timeout | undefined = setTimeout(() => {
execPromise.resolve();
traceError(`Shell execution timed out: ${command.executable} ${command.args?.join(' ')}`);
}, 2000);
```
in src/features/terminal/terminalActivationState.ts

We could do a Promise race where we resolve faster, via listening to `.onDidEndTerminalShellExecution` and watching if exitCode returned with values other than `undefined`.

Also we should watch for exitCode regardless to ensure if command failed/succeeded. (For example, in the middle of the activation, what if user interrupted and stopped? We should know this)

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.