3 ms·
I disagree about smaller PRs being always better. I often find them more difficult to review if they contain only small part of the feature - I see something an
by The_Colonel 3y ago
I disagree about smaller PRs being always better. I often find them more difficult to review if they contain only small part of the feature - I see something and ask - why is this like this? Oh, it's used like that in a subsequent PR... which then forces me to look up that second PR and review them together anyway.
It also brings significant additional overhead - generally every merge to master has to be tested individually, since there's no guarantee all PRs will be merged before release goes out.