Let's co-create a Blueprint review checklist for the future reviewers. Here's the initial draft:
How to get started?
- Subscribe to all the new PRs and issues in this repository to get notifications about new submissions
What to review?
- New Pull Requests without any labels
- Pull Requests with a "needs review" label
How to review?
What to do next?
If the answer to all these questions is "yes", you can just merge the Pull Request. The gallery will be automatically refreshed by a CI job. You're done!
If the answer to any of these questions is "no":
- Leave feedback for the person submitting the Pull Request.
- Tell them exactly what's wrong and give them tips on how to improve it. Over time we should accumulate a nice "copy&paste" database of those tips to streamline the process.
- If they reply, re-review.
- If they don't reply within 14 days, close the PR.
This simple workflow may suffice. If it doesn't, let's explore:
- Automatically adding "awaiting review" and "needs changes" labels when the PR is created and commented on
- A project board with columns representing PRs with those labels
Let's co-create a Blueprint review checklist for the future reviewers. Here's the initial draft:
How to get started?
What to review?
How to review?
What to do next?
If the answer to all these questions is "yes", you can just merge the Pull Request. The gallery will be automatically refreshed by a CI job. You're done!
If the answer to any of these questions is "no":
This simple workflow may suffice. If it doesn't, let's explore: