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
Maintainers already get points for either closing a pull request or merging them in, and this has historically been enough because most of time a pull review is only ever looked at by one maintainer.
But should we make it so reviews also get a point for situations like #32 where a bunch of reviewers submitted a review? Having a point for submitting review would solve this situation of reviewers-that-aren't-the-closer/merger get credit, but then our normal reviewer-and-closer/merger situation means people will be doubling up on maintainer points.
The text was updated successfully, but these errors were encountered:
In prinicple sounds good to me too, but I am concerned this would be used to "game" the leaderboard. By default, anyone can add reviews, so one could just approve "everything everywhere all at once" without any substance to it.
Maintainers already get points for either closing a pull request or merging them in, and this has historically been enough because most of time a pull review is only ever looked at by one maintainer.
But should we make it so reviews also get a point for situations like #32 where a bunch of reviewers submitted a review? Having a point for submitting review would solve this situation of reviewers-that-aren't-the-closer/merger get credit, but then our normal reviewer-and-closer/merger situation means people will be doubling up on maintainer points.
The text was updated successfully, but these errors were encountered: