News for hackers. Stuff that matters.
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.
Re:Ask HN: How would you improve GitHub? by tedmiston on Monday June 27, 2016 @09:11PM (#11990795)
What do you mean? Do you normally have pages and pages of pull requests at one time?
Re:Ask HN: How would you improve GitHub? by Jemaclus on Tuesday June 28, 2016 @12:26PM (#11995072)
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.")

Re:Ask HN: How would you improve GitHub? by tedmiston on Tuesday June 28, 2016 @10:10PM (#11999058)
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.

[0]: https://gitlab.com/gitlab-org/gitlab-ce/issues/14103

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.