Skip to content

Commit b77d6bb

Browse files
authored
docs: see PR review guidelines (#74)
1 parent 21ccbd3 commit b77d6bb

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

docs/CONTRIBUTING.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,17 @@ before that, report the time you have spent on it.
290290
> [!NOTE]
291291
> When contributing, it's essential to report time accurately, including all stages of development (planning, implementation, QA). We encourage opening a PR at the start of your work, even during the planning or investigation phase. Programming and designing isn't just about writing code or creating designs; it also involves planning (40%) and QA (20-30%).
292292
293+
### Reviewing PRs
294+
295+
Use **Request Changes** (reject) only for objective problems:
296+
297+
- PR doesn't solve the stated problem.
298+
- A bug is introduced.
299+
- Code style is inconsistent.
300+
- Required guidelines are violated.
301+
302+
Use **Comment** for optional improvements or suggestions.
303+
293304
### Scout approach
294305

295306
If you ever have free time, be proactive and apply the scout approach: own the

0 commit comments

Comments
 (0)