drifty/comaps
1
0
Fork 0
mirror of https://codeberg.org/comaps/comaps synced 2025-12-19 04:53:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

[docs] clarify politeness

Browse Source 
Signed-off-by: Bastian Greshake Tzovaras <bgreshake@googlemail.com>
This commit is contained in:
Bastian Greshake Tzovaras
2025-08-09 11:22:06 -03:00
committed by Harry Bond
parent 0fa12070e0
commit 5a4f10a584
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
docs/PR_GUIDE.md
View File

@@ -20,7 +20,7 @@ This document gives some guidelines to write and review PR with essential elemen
## Review a Pull Request (PR): ## Review a Pull Request (PR):
- All review comments should be polite and concern the PR - We expect all comments to be polite, including for PR reviews. For PRs feedback should also only concern the PR
- It's better to ask the developer to make the code simpler or add comments to the codebase than to understand the code through the developer's explanation - It's better to ask the developer to make the code simpler or add comments to the codebase than to understand the code through the developer's explanation
- If a developer changes the PR significantly, the reviewers who have already approved the PR should be informed about these changes - If a developer changes the PR significantly, the reviewers who have already approved the PR should be informed about these changes
- A reviewer should pay attention not only to the code base but also to the description of the PR and commits - A reviewer should pay attention not only to the code base but also to the description of the PR and commits