[Blocker] Add a PR CI workflow and make tests gate the deploy
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 65/100
調査の方向性
.github/workflows/azure-app-service.yml の91-92行付近を読み、LearnStack.sln と LearnStack.csproj のビルドターゲットを確認することから始めます。restore、solution build、dotnet test を使用して、pull request と push 用の .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分
- マージ済み PR（30日）
- 9
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
kasuken/LearnStack のほかの issue
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
kasuken/LearnStack#59 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
kasuken/LearnStack#72 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
kasuken/LearnStack#71 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
kasuken/LearnStack#70 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 45/100
kasuken/LearnStack#69 ·
メンテナーはふだん 1 日以内に返信
kasuken/LearnStack の issue をすべて見る
似ている issue
-
from:ci priority:P0
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
spec-kitty/spec-kitty#6621 ·
メンテナーはふだん 1 日以内に返信
-
ci enhancement good first issue level1
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
FrostByte-49/Wharf#56 ·
-
ci: workflow files drifted from upstream templates対応中かも @jackgdsf が今日担当しました。 オープンpdk pdk:ihp
難易度 1/5 1時間未満 初心者へのやさしさ 86/100
gdsfactory/IHP#284 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 61/100
seaweedfs/seaweedfs#11963 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
メンテナーはふだん 2 日以内に返信