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

ci: pr に assignees と reviewers を自動付与 #65

Merged
merged 1 commit into from
Dec 30, 2022

Conversation

kinari321
Copy link
Contributor

@kinari321 kinari321 commented Dec 12, 2022

全体的な意図 ( What, How )

  • Pull Request を作成した際に、レビュアーと担当者を自動でアサインできるようにするため

追加・変更の文脈、位置づけ。なぜこの追加・変更が必要か ( Why )

  • 後から Pull Request を見返した時に担当者がわかりやすくなる
  • Pull Request をオープンにしたとき、レビューを出す依頼を忘れることを防ぐため

この PR がマージされると何が嬉しいか

  • 担当者とレビュアーが自動アサインされることで、手動作業が減る

Checklist

追加・変更したテストケースの見出し一覧

  • 特に無し

Issue

懸念点

参考にしたサイト

レビュアーに手動 QA 作業してほしいですか

  • QA 作業をして欲しい( MUST: QA 手順 )
    • 加えて、記述した手順も見て欲しい
  • QA 作業をしなくてもよい
    • ただ、記述した手順は見て欲しい

QA 作業手順

レビューする時

コメントする時

以下のようなラベルをつけると温度感や、ざっくりと伝えたいことががわかります(小文字でも OK です。厳密な使い分けは不要です)

  • MUST: 必ず修正・変更して欲しい
  • WANT: できれば修正・変更して欲しい
  • IMO: (In my opinion) 私の意見では
  • IMHO: (In my humble opinion) 私のつたない意見では
  • nits: (nitpick) ほんの小さな指摘。インデントミスなどの細かいところに。
  • ASK: 質問。わからないことがあれば質問してみましょう。
  • FYI: (For Your Informatio) 参考までに
  • GOTCHA: やったぜ
  • NP: 問題ない

コピペ用絵文字

https://emojipedia.org/

😀:にっこり
😖:困った
🤔:むむ

👍:(・∀・)イイネ!!

@kinari321 kinari321 self-assigned this Dec 12, 2022
@kinari321 kinari321 linked an issue Dec 12, 2022 that may be closed by this pull request
@kinari321 kinari321 force-pushed the feature-#27/assign-reviewers-assignees branch 2 times, most recently from 62639ac to 20ce310 Compare December 19, 2022 14:04
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@kinari321 kinari321 marked this pull request as ready for review December 24, 2022 06:01
@kinari321 kinari321 requested a review from a team December 24, 2022 06:02
Copy link
Contributor

@sunakan sunakan left a comment

Choose a reason for hiding this comment

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

コメントしましたー

確認お願いします

.github/assign.yml Outdated Show resolved Hide resolved
.github/workflows/assign-reviewers-assignees.yml Outdated Show resolved Hide resolved
.github/assign.yml Outdated Show resolved Hide resolved
.github/assign.yml Show resolved Hide resolved
.github/assign.yml Outdated Show resolved Hide resolved
.github/assign.yml Outdated Show resolved Hide resolved
@kinari321 kinari321 requested a review from a team December 28, 2022 23:25
Copy link
Contributor

@sunakan sunakan left a comment

Choose a reason for hiding this comment

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

コメントしましたー

  • must * 1
  • imo * 1

ですー

確認お願いします

.github/workflows/assign-reviewers-assignees.yml Outdated Show resolved Hide resolved
.github/workflows/assign-reviewers-assignees.yml Outdated Show resolved Hide resolved
@sunakan
Copy link
Contributor

sunakan commented Dec 30, 2022

ありがとうございます。
出来上がったものに対して LGTM です

[nits]

最後にコミットをキレイにまとめてもらっていいですか?
(git rebase 等を利用して 1 or 2つにまとめる)

@kinari321 kinari321 force-pushed the feature-#27/assign-reviewers-assignees branch from 7b07633 to 84c5963 Compare December 30, 2022 21:56
Copy link
Contributor

@sunakan sunakan left a comment

Choose a reason for hiding this comment

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

LGTMです

ありがとうございます

@sunakan sunakan merged commit d4b6f3f into main Dec 30, 2022
@sunakan sunakan deleted the feature-#27/assign-reviewers-assignees branch December 30, 2022 22:09
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.

PR に Reviewers と Assignees を自動でアサイン
2 participants