Re:Ask HN: How would you improve GitHub?
by Jemaclus on Monday June 27, 2016 @04:14PM
(#11989112)
Paginate pull requests. For the love of all that is holy, paginate pull requests. It's probably the main thing that frustrates the hell out of me when doing code reviews. Argh.
[ Parent
| on HN ]
What do you mean? Do you normally have pages and pages of pull requests at one time?
If you have a lot of files or a lot of lines that get changed, eventually it just cuts off and says "Diff is too large" and won't show me the remaining files, so I have to do the diff manually at the command line. It sucks pretty bad.
(Note: our rules suggest small PRs, but occasionally, especially for large rewrites or large features, the PRs can be huge. Lack of pagination makes code reviews extremely difficult, since it just stops showing files, rather than saying "The rest of the files are on the next page.")
Interesting. We use GitLab which doesn't seem to have this limitation that I'm aware of. I've used it [GitLab] for PRs up to a few thousand lines.
Re:Ask HN: How would you improve GitHub?
by jobvandervoort on Wednesday June 29, 2016 @04:36AM
(#12000282)
We actually do show this when the diff is too large, but we're improving this with GitLab 8.10. In 8.10 diffs become collapsible, so you can choose to hide / show very large diffs for performance reasons [0] or to make it easier to walk through diffs with many files.
Re:Ask HN: How would you improve GitHub?
by tedmiston on Wednesday June 29, 2016 @10:54AM
(#12001998)
I stand corrected, in the best possible way. How exactly does GitLab define "large diff" anyway?
Re:Ask HN: How would you improve GitHub?
by jobvandervoort on Wednesday June 29, 2016 @11:37AM
(#12002327)
A large diff has 100 files / 5000 lines in total. A single large file is >=5KB.