• Dev
  • Engineering Practices
  • Pull Request

Pull Request Guide

Proposing changes through pull request / merge request.

Keep The Changes Miniscule as Possible

Change one thing / context at a time. This ensure:

  1. The changes is easily reverted when needed and won't add any complexities.
  2. Ease / speed up the review process as the changes is small and scoped down. Reviewers will also feel more confident to approve smaller and scoped changes rather than big/huge and multiple changes at a time.

Include Related Attachments / References

  • Attach anything that can give reviewers some glimpse of the changes you made. It can be a brief explanation, UI previews.
  • Include any references related to the changes (tickets, sources, etc)
Last updated on November 18, 2022