Skip to content

Conversation

mafreud
Copy link
Contributor

@mafreud mafreud commented Oct 4, 2025

Summary by CodeRabbit

  • チョア
    • 再利用可能なCIワークフローに「ビルド確認」ステップを追加。テスト完了後にビルドを実行し、ビルドが失敗した場合はジョブを失敗として終了します(Firebase Functions/GitHub Apps の各ワークフローが対象)。
    • 既存の手順の順序やエラーハンドリングの挙動に変更はありません。
    • ユーザー向けの機能やUIの変更はありません。

Copy link

linear bot commented Oct 4, 2025

Copy link
Contributor

coderabbitai bot commented Oct 4, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

2つの再利用可能なGitHub Actionsワークフロー(Firebase Functions用とGitHub Apps用)に、テスト後にnpm run buildを実行する「Check build」ステップを追加。既存の手順順序は維持され、ビルド失敗時にはジョブが失敗する。

Changes

Cohort / File(s) Change Summary
Reusable CI Workflows
\.github/workflows/reusable-firebase-functions-ci.yml, \.github/workflows/reusable-github-apps-ci.yml
テスト後に「Check build」ステップ(npm run build)を挿入。その他の手順やエラーハンドリングの変更なし。

Sequence Diagram(s)

sequenceDiagram
  actor Dev as Developer
  participant GH as GitHub Actions Runner
  Dev->>GH: push / PR
  GH->>GH: Run tests
  GH->>GH: Check build (npm run build)
  alt build fails
    GH-->>Dev: Job fails
  else build succeeds
    GH-->>Dev: Job succeeds
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Refactor CI #461: 同じ再利用可能ワークフロー群の作成/改名・リファクタに関する変更で、本PRの「Check build」追加と同一ファイル群を触っているため関連。

Poem

ぼくはCI原っぱのウサギちゃん
テストの丘をぴょんと越え
こんどはビルドの森へダッシュ!
失敗なら耳ピクり、成功なら尻尾フリフリ
緑の草原、チェックは完了🌿✅

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch op-293

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between a060eaf and 0ad4666.

📒 Files selected for processing (2)
  • .github/workflows/reusable-firebase-functions-ci.yml (1 hunks)
  • .github/workflows/reusable-github-apps-ci.yml (1 hunks)

Comment @coderabbitai help to get the list of available commands and usage tips.

@mafreud mafreud merged commit 7ad1ca0 into develop Oct 4, 2025
5 of 6 checks passed
@mafreud mafreud deleted the op-293 branch October 4, 2025 06:28
@coderabbitai coderabbitai bot mentioned this pull request Oct 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant