Skip to content

Commit

Permalink
Update docs/CONTRIBUTING.md
Browse files Browse the repository at this point in the history
Co-authored-by: Stefan Negru <[email protected]>
  • Loading branch information
aaperis and blankdots authored Dec 20, 2023
1 parent 65bb6cf commit 6602ec3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ Other people may also look at the PR and review the code.

A reviewer's job is to:

- Write polite and friendly comments - remember that it can be tough to have other people criticizing your work, a little kindness goes a long way. This does not mean we should not comment on things that need to be changed of course.
- Write polite and friendly comments - remember that it can be tough to have other people criticizing your work, a little kindness goes a long way. This does not mean we should not comment on things that need to be changed of course, but instead focus on providing constructive criticism on how the work can be improved.
- Read the code and make sure it is understandable
- Make sure that commit messages and commits are structured so that it is possible to understand why certain changes were made.
- Ensure that the test-suite covers the new behavior
Expand Down

0 comments on commit 6602ec3

Please sign in to comment.