microsoft / microsoft/vscode-python-environments

Leverage execution.exitCode for execution promise resolve

オープン
#169 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

@anthonykim1 がすでに取り組んでいます。

2025年2月7日 から。

debt needs PR
主要言語
TypeScript
スター
138
フォーク
62
平均マージ
1日 4時間
マージ済み PR(30日)
35

説明

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)

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。