|
|
|
|
|
by harimau777
255 days ago
|
|
100% agree! The pattern that I've found works well is: Reviewing a PR with "Approval" status means that the comments are just suggestions. Choosing "Comment" status means that the comments are optional but important enough that I want to make sure that you read them. If you come back and say "I read them but decided not to make any changes" then I'm happy to approve the PR. Choosing "Request changes" means that the comments aren't just nits. |
|