|
|
@ -106,7 +106,7 @@ Before starting coding, we should ensure all devs / contributors are aware of: |
|
|
|
Your feedback will often be better received if you pose it in the form of a question. |
|
|
|
Your feedback will often be better received if you pose it in the form of a question. |
|
|
|
|
|
|
|
|
|
|
|
- Pull request should be reviewed to comply to coding best practices. |
|
|
|
- Pull request should be reviewed to comply to coding best practices. |
|
|
|
- You should take the responsability of the PR you are reviewing. |
|
|
|
- You should take the responsibility of the PR you are reviewing. |
|
|
|
- You should make sure the app is viable after the PR is being merged. |
|
|
|
- You should make sure the app is viable after the PR is being merged. |
|
|
|
- You should make sure the PR is correctly tested (e2e tests, unit tests) |
|
|
|
- You should make sure the PR is correctly tested (e2e tests, unit tests) |
|
|
|
- Ideally You should have enough knowledge to be able to fix related bugs. |
|
|
|
- Ideally You should have enough knowledge to be able to fix related bugs. |
|
|
|