Skip to content

Conversation

@nkoji21
Copy link
Owner

@nkoji21 nkoji21 commented Oct 10, 2025

Github Actionsの設定項目を追加

  • timeout 5m で設定
  • concurrency を設定
  • permission を設定

timeout 5m の理由として、フロントエンドのFormatter LinterやBuildのみなら、1m 2m で終わるため

permission

content: read

このサイト勉強になった。
https://gihyo.jp/article/2024/10/good-practices-for-github-actions

@nkoji21 nkoji21 self-assigned this Oct 10, 2025
Copilot AI review requested due to automatic review settings October 10, 2025 16:21
@nkoji21 nkoji21 added 🔧 環境 開発環境の構築や設定に関するタスク 🖥️ フロントエンド ユーザーが直接見る画面や操作に関わるタスク labels Oct 10, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

フロントエンドCIの改善を行い、GitHub Actionsのベストプラクティスに従った設定を追加。

  • timeout設定による無限実行の防止
  • concurrency設定による同時実行制御
  • permissions設定によるセキュリティ強化

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/frontend-ci.yml timeout、permissions、concurrency設定を追加し、引用符を統一
.github/workflows/backend-ci.yml 引用符の統一のみ

- 'client/**'
- '.github/workflows/client-ci.yml'
- "client/**"
- ".github/workflows/client-ci.yml"
Copy link

Copilot AI Oct 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[必須] ワークフローファイル名が一致していません。ファイル名は 'frontend-ci.yml' ですが、paths では 'client-ci.yml' を監視しています。正しいファイル名 '.github/workflows/frontend-ci.yml' に修正してください。

Suggested change
- ".github/workflows/client-ci.yml"
- ".github/workflows/frontend-ci.yml"

Copilot uses AI. Check for mistakes.
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on November 12

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@nkoji21
Copy link
Owner Author

nkoji21 commented Oct 10, 2025

wwwwwwwww
approve早すぎて好き

@nkoji21 nkoji21 merged commit c195c5e into main Oct 10, 2025
2 checks passed
@nkoji21 nkoji21 deleted the refac/front-ci branch October 10, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 環境 開発環境の構築や設定に関するタスク 🖥️ フロントエンド ユーザーが直接見る画面や操作に関わるタスク

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants