This document describes some processes and guidelines we follow as a team and will help make the contribution process as smooth as possible! ✨
a) Follow the PR and Code Review Culture at Pleo guidelines
When creating your PR, please add the following reviewers:
@pleo-io/team-accounts-payable
Keep the number of reviewers small to make it obvious who is responsible for reviewing the PR.
Unless otherwise arranged, we will never merge your PRs. It's your code and you know best when it's ready to be merged.
Obviously this does not apply on weekends and days we are out of office.
GitHub will avoid auto-assigning organisation members that are set to Busy, so watch out for the status indicator when assigning reviewers. 🏖
In general, we will review and approve/leave feedback on your PR within 24 hours, but in some cases we may not be able to dedicate the time necessary to give it a thorough review. Feel free to ping us if you see PRs go stale.
Keeping PRs focused on a single change makes them easier (and faster) to review. We may ask you to split up a large PR if it includes many unrelated changes.
The more context you can provide in the PR description, the better equipped we will be to review it. This will make reviews more effective and faster. 🚀
You can also add inline comments on your own PR to add some context or point our areas you want your reviewers to focus on.
Thank you for taking the time to read this document!
We hope it has been helpful!
If you have any suggestions or feedback please reach out to us or submit a PR changing these guidelines. ❤️