א
PR קטן עדיף על PR גדול. הפותח PR של אלף שורות — גורם לביקורת שטחית, ולרוב יאושר מתוך עייפות ולא מתוך הבנה.
אמרו חכמים: reviewer שראה אלף שורות — כבר לא קורא בשורה חמש מאות. הוי אומר: PR קטן הוא מתנה לעמיתיך, ומי שנותן מתנות — אהוב.
מחלוקת
רבי ג'ורג' הקטן אומר:
PR חייב להיות קטן — מאה שורות לכל היותר. יותר מזה — פצל. reviewer שרואה אלף שורות כבר לא קורא.
רבי לינוס הנורדי חולק ואומר:
Feature הוא feature. אי אפשר לפצל driver של kernel ל-PR של מאה שורות, ואי אפשר לפצל refactor גדול לחלקים שכל אחד שובר את הקוד.
רבי ג'ורג' הקטן:
אפשר תמיד לפצל — אם חושבים מספיק.
רבי לינוס הנורדי:
חשבת פעם על שינוי ממשק שמשפיע על מאתיים קבצים?
רבי ג'ורג' הקטן:
feature flag, backwards compatibility — יש פתרונות.
לרוב — קטן. לפעמים — גדול אך מוסבר היטב. תמיד — ברור ומכוון.