Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

[Blocker] Add a PR CI workflow and make tests gate the deploy

未关闭
#42 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
65/100
Issue 类型
功能
描述清晰度
描述清楚
活跃度
活跃
技术栈
azure, csharp, github-actions
领域
ci-cd, cloud, devops

调研方向

先阅读 .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 内容生成。

描述

enhancement

Tier 1 — release blocker

Problem

Three separate gaps:

  1. The deploy workflow runs tests with continue-on-error: true, so a failing test never blocks a production deploy.
  2. It runs dotnet test --no-build at solution level after building only LearnStack.csproj, so the test project is likely never built and the tests probably do not execute at all — the swallowed error hides this.
  3. There is no PR build workflow at all: the only workflow triggers on push to main and deploys straight to production.
Evidence
  • .github/workflows/azure-app-service.yml:91-92
  • .github/workflows/ contains only azure-app-service.yml
Proposed fix
  1. Add .github/workflows/ci.yml: restore, build LearnStack.sln, dotnet test on pull_request and push.
  2. Mark it a required status check on main.
  3. 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 模板
  • 阅读贡献指南

从这里开始

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

kasuken/LearnStack 的其他 Issue

查看 kasuken/LearnStack 的全部 Issue

相似的 Issue

更多 DevOps Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。