Re:The ideal PR is 50 lines long
by wubrr on Wednesday February 07, 2024 @04:38PM
(#39294646)
100% agree, if you have multiple commits to complete the change during development, just squash them into one for the PR.
That being said many companies (including some FAANG) use commit counts as a performance metric, thereby directly disincentivizing clean, readable commits.
That being said many companies (including some FAANG) use commit counts as a performance metric, thereby directly disincentivizing clean, readable commits.