full-day workshop: Correct the CI workflow triggers and first-run verification

未关闭

#264 创建于 2026年9月14日 1 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 3/5
预计耗时: 1-2 天
新手友好度: 76/100

Issue 类型: 文档
描述清晰度: 描述清楚
活跃度: 活跃
技术栈: github-actions
领域: ci-cd, documentation

调研方向

从 content/full-day/4-testing.md 开始，并将其中对触发器的说明与 .github/workflows/server-test.yml 进行比较。在一个全新的模板仓库中验证首次运行流程，包括 workflow 变更、server 变更、pull request 以及按需执行。文档中记录的事件和验证步骤与观察到的 GitHub Actions 运行结果一致，即视为完成。

由索引模型根据 Issue 内容生成。

描述

priority: P0

Problem

content/full-day/4-testing.md describes merge as a GitHub Actions event, but there is no merge workflow event; a merge to main is normally handled as a push. Learners are also told to request an on-demand trigger, while the example omits workflow_dispatch.

The example restricts both push and pull_request to app/server/**. The first commit adds only .github/workflows/server-test.yml, so the text's claim that this push triggers the workflow is false: the path filter prevents that first run. Learners can reach the Actions page with no run to inspect.

Suggested change

Teach the actual event semantics and provide an example whose triggers support every verification step in the exercise.

Acceptance criteria

merge is removed as an event name and explained as a push to the target branch.
The example includes workflow_dispatch when on-demand execution is required.
Adding or changing the workflow itself triggers a run, or learners are explicitly told to dispatch it manually.
Server changes still trigger push and pull-request runs.
The exact first-run flow is tested in a fresh template repository.

Related: #236 separately tracks the pytest/unittest inconsistency.

主要语言: Python
星标: 80
派生: 161
平均合并: 31 分钟
30 天内合并 PR: 1

贡献指南

打开贡献指南

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

github-samples/pets-workshop 的其他 Issue

full-day workshop: Add Azure prerequisites, cost warning, and resource cleanup 未关闭

priority: P0

难度 2/5 1-2 天新手友好度 78/100

github-samples/pets-workshop#268 · 1 条评论 · 2026年9月14日创建
full-day workshop: Add the missing commit step before creating the feature pull request 未关闭

priority: P0

难度 1/5 1-3 小时新手友好度 88/100

github-samples/pets-workshop#267 · 1 条评论 · 2026年9月14日创建
1-hour workshop: Add concrete validation scenarios for the filter feature 未关闭

priority: P1

难度 1/5 1-3 小时新手友好度 92/100

github-samples/pets-workshop#261 · 1 条评论 · 2026年9月14日创建
1-hour workshop: Include the page data owner in the filter exercise context 未关闭

priority: P1

难度 2/5 1-3 小时新手友好度 78/100

github-samples/pets-workshop#257 · 1 条评论 · 2026年9月14日创建
GitHub security workshop: Add security overview, governance, and rollout exercise 未关闭

priority: deferred

难度 4/5 3-5 天新手友好度 45/100

github-samples/pets-workshop#278 · 1 条评论 · 2026年9月14日创建

查看 github-samples/pets-workshop 的全部 Issue

相似的 Issue

slack-gif-creator: validate_gif() multiplies the last frame's duration by the frame count, so reported duration/fps are wrong (1.0s GIF reported as 1.8s) 未关闭

难度 2/5 1-3 小时新手友好度 75/100

anthropics/skills#1811 · 1 条评论 · 2026年9月23日创建
App fails to start with gradio 6 (allowed by `gradio>=5.49.1`) 未关闭

难度 2/5 1-3 小时新手友好度 75/100

speaches-ai/speaches#678 · 2026年9月23日创建
ModuleNotFoundError: No module named 'mcp.server.fastmcp'. 未关闭

bug

难度 2/5 1-3 小时新手友好度 75/100

datalayer/mcp-compose#42 · 2026年9月23日创建
import spacy fails with ModuleNotFoundError: No module named 'click' 未关闭

难度 2/5 1-3 小时新手友好度 75/100

conda-forge/spacy-feedstock#177 · 2026年9月23日创建
humaneval: extract_function_body corrupts a fenced body-only reply, so the same correct code scores INCORRECT fenced and CORRECT unfenced 未关闭

难度 2/5 1-3 小时新手友好度 70/100

UKGovernmentBEIS/inspect_evals#2523 · 2026年9月23日创建

更多 Python Issue