Pull Requests
This document serves as a guide for how you might review pull requests.
Use your best judgement when reviewing PRs, and most of all remember to be kind, friendly, and encouraging when responding to users. Many users are new to open source and/or typed linting. It's imperative we give them a positive, uplifting experience.
If you're ever unsure on any part of PR reviews, don't hesitate to loop in a maintainer that has more context to help!
PR Flow
We include a set of common responses to PRs in .github/replies.yml, intended to be used with the Refined Saved Replies extension.
Don't treat these as exact responses you must use: they're just a starting copy+paste helper.
Please do adopt your specific responses to your personal tone and to match the thread for non-straightforward PRs.
TODO: This will be filled out... soon!
Pruning Old PRs
Every so often, we like to search for open PRs awaiting response to find ones that might have been forgotten.
Our flow for PRs that have been waiting for >=1 month is:
- Ping the author with a message like the Checking In template
- Add the
stalelabel to the PR - Wait 2 weeks
- If they still haven't responded, close the PR with a message like the Pruning Stale PR template
Suggestions
Consider adding fun to PR reviews with GIFs. The benefits of using GIFs in PR reviews are that it adds a touch of humor and lightheartedness to the process, builds camaraderie and team spirit, and can help break the ice by making users feel welcomed. Please remember to use appropriate and respectful GIFs and use them sparingly; GIFs should enhance the conversation, not detract from it.
We like to use the GIFs for Github extension: available as GIFs for GitHub (Chrome) and GIFs for GitHub (Firefox). ✨