I’ve been triaging some open PRs, and I wonder if we should consider updating some of the labels to simplify triaging actionable PRs.
In other projects that I maintain, I rarely look at a PR that does not have a green checkmark (unless someone @ mentions me). Once the green check appears, I know that the PR is ready for review and subsequent merge if there are no review issues.
Currently, many PRs get a red X when there is no news or issue number or if the CLA needs signature. Yet, those same PRs get an “Awaiting …” label. Ideally, I would like to filter the PRs that are complete and actionable (ready to merge on positive review).
Perhaps “news” and “issue number” shouldn’t trigger the red X. Alternatively, waiting to apply any “Awaiting…” label until the tests and news, issue #, and CLA all pass may also be a good approach. Since we are tight on reviewing time, it increases our productivity to only flag for human review the PRs that have met all automated checks.