Re:Give some love to your PR
by noname120 on Tuesday July 14, 2020 @04:25PM
(#23837355)
I keep seeing this “keep it small” mantra everywhere. There is a good reason people do large pull requests instead of multiple dependent pull requests. The reason is that current UIs are atrocious. For example GitHub plain doesn't support them which makes it an absolute nightmare to review them (irrelevant changes[1]).
Additionally, syncing the different branches with Git is a pain, especially when you have several levels of dependencies.
[1] I'm well aware of the fact that you can set the parent branch to something else than master. But in that case, you'll have to merge the parent branch after the branches which depend on it. This completely defeats the purpose of splitting pull requests.
Additionally, syncing the different branches with Git is a pain, especially when you have several levels of dependencies.
[1] I'm well aware of the fact that you can set the parent branch to something else than master. But in that case, you'll have to merge the parent branch after the branches which depend on it. This completely defeats the purpose of splitting pull requests.