You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/09/27 16:05:22 UTC

[GitHub] [tvm] leandron commented on issue #9057: [VOTE] Adopt round-robin assignment of reviewers for GitHub pull request reviewer assignment.

leandron commented on issue #9057:
URL: https://github.com/apache/tvm/issues/9057#issuecomment-928023086


   -1
   
   I think we should try other measures before an enforced round-robin, such as:
   - tuning the list of maintainers in a more detailed way, with perhaps less people per directory so that lots of people don't get e-mails for simple reviews
   - providing more guidance to incentive smaller, incremental patches, which won't touch many areas of the code at once, therefore reaching out to less reviewers
   
   Other points:
   
   1. If people feel like they are getting too many *e-mails* for reviews, GitHub e-mails are very descriptive, and people can filter them easily i.e. there is always the reason on why you're getting involved such as "Review requested" (the ones added automatically), "Mention" (if you're quoted by name), "Comment" (on the threads you comment).
   
   2. How to match automatic assignment with the time people can dedicate to reviews, which can be variable in different periods of the year (holidays, deadlines, other priorities, personal, etc...). On this point, I think it would be fair to have a documented way for people to decline assigned reviews without necessarily specifying a reason.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org