Pull Request Practices

If you've been added as a collaborator to the repository, please do not push unless the commit you are making is trivial i.e. doesn't require a code review. If you're unsure about this, please ask!

• Please run static code analysis tools locally before you make a pull request, don't rely on Codeclimate to do that for you.
  • Frontend Practices for Code Quality
  • Backend Practices for Code Quality
• Make sure you run the test suite locally before you make a pull request, don't rely on CircleCI to do that for you.
  • Automated Testing
• Make sure commit messages are clear and concise are tagged with the issue number e.g. "[#99] Fixes some sample issue".
• The following pull request template will appear when you create your PR. Please follow the instructions there!
• As a pull request (PR) reviewer, if you think the PR is good to go (including passing tests) make sure to comment with LGTM (looks good to me).
• You can either merge it yourself or tell the PR creator to do it themselves once approved.

Everyone is encouraged to participate in code reviews, so please do so! Check out our Code Review Practices!