You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When I have Declined a review, it's a little misleading to see a red Last Reviewer annotation. I'm not considered in the list of reviewers blocking a PR, so I'm not really the last reviewer. Perhaps this annotation should be left off when Declined.
It's less clear to me how to handle mere Optional reviewers. They're not blocking, but they're still people considered in the list of reviewers. I think my preference is to exclude all Optional reviewers, whether Declined or not, as the red Last Reviewer annotation seems intended to prevent PRs from being blocked.
The text was updated successfully, but these errors were encountered:
kentyman23
added a commit
to kentyman23/azdo-userscripts
that referenced
this issue
Nov 6, 2023
When I have Declined a review, it's a little misleading to see a red Last Reviewer annotation. I'm not considered in the list of reviewers blocking a PR, so I'm not really the last reviewer. Perhaps this annotation should be left off when Declined.
It's less clear to me how to handle mere Optional reviewers. They're not blocking, but they're still people considered in the list of reviewers. I think my preference is to exclude all Optional reviewers, whether Declined or not, as the red Last Reviewer annotation seems intended to prevent PRs from being blocked.
The text was updated successfully, but these errors were encountered: