1-hour workshop: Fail startup clearly when dependencies or services do not start
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 55/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
調査の方向性
app/scripts/start-app.sh と app/scripts/start-app.ps1 から始め、依存関係のインストール、固定された 5 秒間の待機、サービス URL の出力、クリーンアップの経路を追跡します。両方の起動スクリプトを実行し、Python または Node のインストール失敗と Flask または Astro の起動失敗をシミュレートします。完了の条件は、失敗時の終了ステータスが 0 以外で明確であること、管理対象のプロセスが一緒に停止すること、両方のサービスの準備が完了してからのみ URL が表示されること、そして Ctrl+C または部分的な起動後でもクリーンアップが機能することです。
索引モデルが issue の本文から書いたものです。
説明
Problem
The learner setup depends entirely on the startup scripts, but failure handling can report a misleading success state. In app/scripts/start-app.sh, npm install is not checked before npm run dev starts. In app/scripts/start-app.ps1, a failed Node dependency installation emits a warning and continues. Both scripts wait a fixed five seconds and then print both service URLs without checking that Flask and Astro are ready.
This makes common setup failures look like browser or workshop problems and gives independent learners little actionable feedback.
Suggested change
Make both startup scripts stop on dependency installation failures and verify service/process readiness before announcing success.
Acceptance criteria
- A failed Python or Node dependency install exits nonzero with a clear message.
- Failure of either Flask or Astro stops the other managed process.
- URLs are printed only after both services are ready.
- Cleanup still works on Ctrl+C and partial startup failures.
- macOS/Linux and Windows paths have equivalent behavior.
- 主要言語
- Python
- スター
- 80
- フォーク
- 161
- 平均マージ
- 31分
- マージ済み PR（30日）
- 1
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github-samples/pets-workshop のほかの issue
-
priority: P0
難易度 2/5 1〜2日 初心者へのやさしさ 78/100
github-samples/pets-workshop#268 · コメント 1 件 ·
-
priority: P0
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
github-samples/pets-workshop#267 · コメント 1 件 ·
-
priority: P1
難易度 1/5 1〜3時間 初心者へのやさしさ 92/100
github-samples/pets-workshop#261 · コメント 1 件 ·
-
priority: P1
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
github-samples/pets-workshop#257 · コメント 1 件 ·
-
priority: deferred
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
github-samples/pets-workshop#278 · コメント 1 件 ·
github-samples/pets-workshop の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
anthropics/skills#1811 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
speaches-ai/speaches#678 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
datalayer/mcp-compose#42 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
conda-forge/spacy-feedstock#177 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
UKGovernmentBEIS/inspect_evals#2523 ·