Skip to content

Latest commit

 

History

History
12 lines (10 loc) · 921 Bytes

self-review.md

File metadata and controls

12 lines (10 loc) · 921 Bytes

Self-review

You should always review your own PR first before assigning a reviewer to it. Below you can find a checklist of things you should check before submitting your PR.

  • Confirm that the changes solve the issue you are trying to solve partially or fully.
  • Review the code in terms of the OWASP top 10 security issues.
  • Verify that your unit tests are running successfully.
  • Verify that your code follows the style guide. Prettier will take care most of the formatting, but the guide specifies some things regarding documentation.
  • Review your changes by yourself before assigning a reviewer to the pull request. This will help you spot issues like typos, content that doesn't follow the style guide or content that is added by mistake.
  • If there are any failing checks in your PR, troubleshoot them until they're all passing.