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

レビュアー指定をする #1525

Open
Goryudyuma opened this issue Apr 3, 2024 · 2 comments
Open

レビュアー指定をする #1525

Goryudyuma opened this issue Apr 3, 2024 · 2 comments

Comments

@Goryudyuma
Copy link
Member

今、レビュアーを指定していないので、自動的にCODEOWNERがレビュアーになる。
でも実際に見るべきは、PRを作った人であるべき。
(Assigneesに既に入っているのは理解している)
もしレビュアーを、今Assigneesに入れているPRの作者にすれば、CODEOWNERはレビュアーから外れて、いい感じになるのでは?
(もしかしたらCODEOWNERSに加えて、Assigneesもレビュアーになる、という仕様かもしれない?要検証)

@massongit
Copy link
Member

#1530
#1533

修正PR作成直後にレビュアーをアサインする形にしてみたものの、CODEOWNERがレビュアーにアサインされるのは変わらなかった。

@Goryudyuma
Copy link
Member Author

多分PRを作ると同時にレビュアーを指定しないといけないと思う。
一旦レビュアーなしでPRが作られると、そのタイミングでCODEOWNERがレビュアーになってしまうはず。
その後でレビュアーを指定しても結局、って感じ。

もしくは、同じタイミングで作れなさそうであれば、draftで一旦作って、そこでレビュアーを指定、その後openしたら、openされた時にCODEOWNERが指定されるんだろうか。要検証。

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 a pull request may close this issue.

2 participants