1-hour workshop: Fail startup clearly when dependencies or services do not start
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 55/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
调研方向
从 app/scripts/start-app.sh 和 app/scripts/start-app.ps1 开始，跟踪依赖安装、固定的五秒等待、服务 URL 输出以及清理路径。运行两个启动脚本，同时模拟 Python 或 Node 安装失败，以及 Flask 或 Astro 启动失败；完成的标准是：失败时返回非零且明确的状态码，受管理的进程一起停止，只有在两个服务都就绪后才显示 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 分钟
- 30 天内合并 PR
- 1
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 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
-
essnmx good first issue
难度 1/5 1 小时以内 新手友好度 95/100
-
难度 2/5 1-3 小时 新手友好度 65/100
syfoud/Simulated_Scepter#174 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
Giskard-AI/giskard-oss#2840 · 1 条评论 ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success 未关闭area: repo bug perceived difficulty: 2
难度 2/5 1-3 小时 新手友好度 70/100
-
难度 2/5 1-3 小时 新手友好度 75/100
yeti-platform/yeti#1380 ·