Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

プルリクエスト提出時の自動アサインワークフロー作成 #31

Merged
merged 1 commit into from
Dec 26, 2023

Conversation

blendthink
Copy link
Member

概要

レビュー観点

  • 設定ミスがあれば教えてください
  • すでに vars.REVIEWERS には値が入るように設定しています
  • チームをレビュアーに設定できるようにすることも検討しましたが、特殊な権限のトークンが必要になり手間が増えてしまうため、テンプレートプロジェクトでは設定しないようにしました
  • 外部からのプルリクエスト時にも動作するように pull_request_target で起動することも検討しましたが、現状のアプリ開発ではコードを公開することがないため、いったん pull_request で起動するようにしました。

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

備考

特にありません。

Copy link

Ready for review 🚀

@blendthink
Copy link
Member Author

@iseruuuuu @trm11tkr
レビューお願いします 🙏

Copy link
Contributor

@trm11tkr trm11tkr left a comment

Choose a reason for hiding this comment

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

LGTM!

@blendthink blendthink merged commit ae77239 into main Dec 26, 2023
1 check passed
@blendthink blendthink deleted the feature/auto-assign branch December 26, 2023 05: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