All everything aside, reviewing big pull requests on GitHub became nearly impossible - even with the simplest change view it makes you spend too much time on waiting for the page to load the necessary file first. The performance degraded significantly from what was the experience from 10 years ago. UI became an absolute mess. Maybe even vibe-coded.

▲

bjackman 4 hours ago | parent | next [-]

Is there a good code review tool out there? The best one I've used is Gerrit, at least it has a sensible design in principle. Aside from that I've only used GitHub and Gitlab which both seem like toys to me. (And mailing lists, lol).

But the implementation of Gerrit seems rather unloved, it just seems to get the minimal maintenance to keep Go/Android chooching along, and nothing more.

	▲	seabrookmx an hour ago \| parent [-]
		My old job used Gerrit, and new job uses Gitlab. I really miss the information density and workflow of Gerrit. We enforce fast forward merges and squashing for MR's anyways, so we just have an awkward version of what Gerrit does by default. Gitlab CI is good but we use local (k8s-hosted) runners so I have to imagine there's a bunch of options that provide a similar experience.

▲

viraptor 3 hours ago | parent | prev | next [-]

While I'm not annoyed by the slowdown that much, what made me not trust them anymore is being careless with the system. For example I did a review recently on a PR where the collapsing sections were not visible and made 2 patch fragments look like continuous code. I commented that this makes no sense and won't run... only to look like an idiot later. Fortunately the issue was fixed by the time I looked at the PR again, but still, much less trust now.

▲

razighter777 3 hours ago | parent | prev | next [-]

Quick tip: If you type .patch after the PR url it gives you a git patch. Do curl <github patch> | git am and you can apply and review it locally.

	▲	IshKebab an hour ago \| parent [-]
		No need for that. Just install the VSCode GitHub extension and you can just directly open them. It even supports comments. Hell even if you don't use VSCode there are much better options than messing around with patch files.

▲

arccy 4 hours ago | parent | prev [-]

They even have the gall to call it an improved UI for reviewing large pull requests. They must have let UI designers who've never written code before design it.

▲

shimman 4 hours ago | parent | next [-]

IME those types of UI designers are way way way way more common, very few designers seem to understand the platform they are designing on and care more about aesthetics rather than proper platform designs.

▲

bflesch 4 hours ago | parent | prev [-]

That's what happens when the whole company uses high-end macbooks and nobody has an older PC. It's been noted thousands of times on HN but these US companies make money head over fist and do not give a single damn about people on "lower" end devices.

	▲	plorkyeran 4 hours ago \| parent [-]
		Large GitHub PRs are miserably slow even with a maxed out Mac Studio on gigabit fiber with single-digit ms ping to their server. It’s not an example of something that works well on high-end hardware but scales down poorly.