[Blocker] Add a PR CI workflow and make tests gate the deploy
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 65/100
调研方向
先阅读 .github/workflows/azure-app-service.yml 第 91-92 行附近的内容,并检查 LearnStack.sln 和 LearnStack.csproj 的构建目标。使用 restore、solution build 和 dotnet test 为 pull requests 和 pushes 添加 .github/workflows/ci.yml,然后更新 deploy workflow。完成的标准是:测试失败时,必需的 PR 检查和 deploy 测试会明确失败,并且 workflow summary 会显示结果。
由索引模型根据 Issue 内容生成。
描述
Tier 1 — release blocker
Problem
Three separate gaps:
- The deploy workflow runs tests with
continue-on-error: true, so a failing test never blocks a production deploy. - It runs
dotnet test --no-buildat solution level after building onlyLearnStack.csproj, so the test project is likely never built and the tests probably do not execute at all — the swallowed error hides this. - There is no PR build workflow at all: the only workflow triggers on push to
mainand deploys straight to production.
Evidence
.github/workflows/azure-app-service.yml:91-92.github/workflows/contains onlyazure-app-service.yml
Proposed fix
- Add
.github/workflows/ci.yml: restore, buildLearnStack.sln,dotnet testonpull_requestandpush. - Mark it a required status check on
main. - In the deploy workflow, build the solution and drop
continue-on-error.
Acceptance criteria
- A PR with a failing test cannot be merged
- A failing test blocks the deploy job
- Test results are visible in the workflow summary
- 主要语言
- HTML
- 星标
- 10
- 派生
- 0
- 平均合并
- 17 分钟
- 30 天内合并 PR
- 9
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
kasuken/LearnStack 的其他 Issue
-
enhancement
难度 1/5 1 小时以内 新手友好度 91/100
kasuken/LearnStack#59 ·
维护者通常 1 天内回复
-
enhancement
难度 5/5 一周以上 新手友好度 35/100
kasuken/LearnStack#72 ·
维护者通常 1 天内回复
-
enhancement
难度 5/5 一周以上 新手友好度 30/100
kasuken/LearnStack#71 ·
维护者通常 1 天内回复
-
enhancement
难度 5/5 一周以上 新手友好度 35/100
kasuken/LearnStack#70 ·
维护者通常 1 天内回复
-
enhancement
难度 5/5 一周以上 新手友好度 45/100
kasuken/LearnStack#69 ·
维护者通常 1 天内回复
查看 kasuken/LearnStack 的全部 Issue
相似的 Issue
-
area: client bug
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
OCIRepository: pick the Helm chart layer automatically when the first layer is a provenance file未关闭
难度 2/5 1-3 小时 新手友好度 78/100
fluxcd/source-controller#2198 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
e2e-failure ready-to-code
难度 1/5 1 小时以内 新手友好度 82/100
redhat-developer/rhdh-plugin-export-overlays#4485 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100