I have hard times reviewing PR of more than 200 lines. #231

Merged

ChristopheNan merged 1 commit from mdk-for-shorter-prs into 3.11

2023-11-25 16:57:43 +01:00

mdk commented

2023-11-25 15:42:49 +01:00

Owner

Smaller PR are:

Faster to review: reviewers may not have many free hours straight.
Easier to merge: if a translator does a lot of work, better have
part of it merged quickly (by merging good PRs) instead of blocking
the whole work due to a small detail.

Smaller PR are: - Faster to review: reviewers may not have many free hours straight. - Easier to merge: if a translator does a lot of work, better have - part of it merged quickly (by merging good PRs) instead of blocking - the whole work due to a small detail.

mdk added 1 commit

2023-11-25 15:42:49 +01:00

I have hard times reviewing PR of more than 200 lines. b846b2d587

Smaller PR are:

- Faster to review: reviewers may not have many free hours straight.
- Easier to merge: if a translator does a lot of work, better have
- part of it merged quickly (by merging good PRs) instead of blocking
- the whole work due to a small detail.