Skip to content

revert: "ci(workflows): consolidate pr workflows into ci.yml" - #16

Merged
Yusuke4869 merged 1 commit into
mainfrom
revert-15-ci/consolidate-pr-workflows
Mar 7, 2026
Merged

Yusuke4869 merged 1 commit into
mainfrom
revert-15-ci/consolidate-pr-workflows

Conversation

@Yusuke4869

@Yusuke4869 Yusuke4869 commented Mar 7, 2026 •

Copy link
Copy Markdown
Contributor

Reverts #15

Summary by CodeRabbit

リリースノート

  • Chores
    • ワークフロー設定を再構成。PR title検証ワークフローを有効化し、PRの自動割り当て機能を最適化しました。

Copilot AI review requested due to automatic review settings March 7, 2026 15:35
@coderabbitai

coderabbitai Bot commented Mar 7, 2026 •

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

GitHub Actions ワークフロー構造が統合から分散型へ変更されました。ci.ymlが削除され、auto-author-assign.ymlとsemantic-pr-title.ymlがコメント状態から有効化されました。PR自動割り当てとセマンティックPRタイトルチェック機能が個別ワークフローに分離されました。

Changes

Cohort / File(s) Summary
ワークフロー有効化
​.github/workflows/auto-author-assign.yml, .github/workflows/semantic-pr-title.yml
コメント記号を削除して両ワークフローを有効化。PR作成者の自動割り当てとセマンティックPRタイトル検証機能がアクティブ化されました。
統合ワークフロー削除
​.github/workflows/ci.yml
PR作成者割り当て、PRタイトルリント検証、失敗時コメント投稿などを含む統合CI ワークフロー全体(85行)を削除。機能は個別ワークフローに分離されました。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 ワークフローが目覚めたよ、
コメント外して独立へ、
統合から分散へシフト、
CI は消えて役割分担、
自動化の新しい道へ!✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRのタイトルは、変更内容を正確に反映しており、PR #15で導入された変更を元に戻すという主な意図を明確に伝えています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-15-ci/consolidate-pr-workflows

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts a previous consolidation of PR-related workflows into a single ci.yml, restoring dedicated workflows for PR title linting and auto-assigning the PR author.

Changes:

  • Re-enabled the standalone Semantic PR Title workflow.
  • Re-enabled the standalone Auto Author Assign workflow.
  • Removed the consolidated .github/workflows/ci.yml.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
.github/workflows/semantic-pr-title.yml Restores PR title lint workflow previously commented out.
.github/workflows/auto-author-assign.yml Restores PR author auto-assign workflow previously commented out.
.github/workflows/ci.yml Deletes the consolidated workflow that previously bundled both jobs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread .github/workflows/semantic-pr-title.yml
Comment thread .github/workflows/semantic-pr-title.yml
Comment thread .github/workflows/auto-author-assign.yml
Comment thread .github/workflows/auto-author-assign.yml

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/auto-author-assign.yml:
- Line 17: Change the external Action reference used in the pull_request_target
job from a mutable tag to an immutable commit SHA: replace the uses entry
"toshimaru/auto-author-assign@v3.0.1" with the repository@<commit-SHA> form
(e.g., toshimaru/auto-author-assign@<SHA>) so the exact commit is executed;
ensure any other external actions in this workflow are likewise pinned to commit
SHAs to prevent upstream tag replacement when running under pull_request_target.

In @.github/workflows/semantic-pr-title.yml:
- Line 22: This workflow uses amannn/action-semantic-pull-request and
marocchino/sticky-pull-request-comment under pull_request_target with write
permissions; replace their tag references with their full commit SHAs (pin to
the exact repository commit) so the executed code cannot change via upstream tag
updates—update the occurrences of "uses:
amannn/action-semantic-pull-request@..." and "uses:
marocchino/sticky-pull-request-comment@..." to use the full SHA strings for each
action.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3cfa8f12-920f-4d4a-9798-2e5076dd6bcb

📥 Commits

Reviewing files that changed from the base of the PR and between e15f52b and 2e3e3f8.

📒 Files selected for processing (3)
  • .github/workflows/auto-author-assign.yml
  • .github/workflows/ci.yml
  • .github/workflows/semantic-pr-title.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/ci.yml
📜 Review details
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2026-02-06T05:30:47.662Z
Learnt from: Yusuke4869
Repo: densanken/PR-sandbox PR: 1
File: .github/workflows/semantic-pr-title.yml:11-13
Timestamp: 2026-02-06T05:30:47.662Z
Learning: amannn/action-semantic-pull-request を使用する場合、WIP 状態の PR を管理するために commit status API (POST /repos/:owner/:repo/statuses/:sha) を利用します。これにはリポジトリのトークンに write:statuses 権限が必要です。該当ワークフロー (.github/workflows/semantic-pr-title.yml のようなファイル) を使用する際は、トークン権限を appropriate に設定し、WIP の PR を 'pending' ステータスへ設定する挙動を理解しておくこと。

Applied to files:

  • .github/workflows/auto-author-assign.yml
  • .github/workflows/semantic-pr-title.yml
🔇 Additional comments (1)
.github/workflows/semantic-pr-title.yml (1)

11-13: statuses: write を戻したのは適切です。

wip: true を有効にした amannn/action-semantic-pull-request では pending status の更新にこの権限が必要なので、この revert で必要権限が復元されています。

Based on learnings amannn/action-semantic-pull-request を使用する場合、WIP 状態の PR を管理するために commit status API (POST /repos/:owner/:repo/statuses/:sha) を利用し、これにはリポジトリのトークンに write:statuses 権限が必要です。

Comment thread .github/workflows/auto-author-assign.yml
Comment thread .github/workflows/semantic-pr-title.yml
@Yusuke4869
Yusuke4869 merged commit 67e320a into main Mar 7, 2026
10 checks passed
@Yusuke4869
Yusuke4869 deleted the revert-15-ci/consolidate-pr-workflows branch March 7, 2026 15:43
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.

2 participants