←back to thread

The GitHub website is slow on Safari

(github.com)
446 points talboren | 2 comments | 27 Aug 25 09:43 UTC | HN request time: 0.206s | source
Show context
muglug ◴[27 Aug 25 13:03 UTC] No.45039093[source]▶
Improvements merged within the last two days by the WebKit team: https://github.com/orgs/community/discussions/170922#discuss...

For my sins I occasionally create large PRs (> 1,000 files) in GitHub, and teammates (who mostly all use Chrome) will sometimes say "I'll approve once it loads for me..."

replies(6): >>45039371 #>>45039546 #>>45039877 #>>45040882 #>>45043276 #>>45050622 #
patrickmay ◴[27 Aug 25 13:26 UTC] No.45039371[source]▶
That seems essentially unreviewable. If you can share without violating an NDA, what kind of PR would involve that many files?
replies(6): >>45039414 #>>45039433 #>>45039455 #>>45039478 #>>45040667 #>>45041353 #
bob1029 ◴[27 Aug 25 13:34 UTC] No.45039478[source]▶
"Upgrade solution from .NET Framework 4.8 => .NET 8"

"Rename 'CustomerEmailAddress' to 'CustomerEmail'"

"Upgrade 3rd party API from v3 to v4"

I genuinely don't get this notion of a "max # of files in a PR". It all comes off to me as post hoc justification of really shitty technology decisions at GitHub.

replies(2): >>45039963 #>>45041641 #
ambicapter ◴[27 Aug 25 14:09 UTC] No.45039963[source]▶
It's not GitHub-specific advice, it's about reviewability of the PR vs. human working memory/maximum attention span.
replies(2): >>45040291 #>>45041083 #
eviks ◴[27 Aug 25 14:34 UTC] No.45040291[source]▶
How much working memory/attention span is required to look through 1000 identical lines "-CustomerEmailAddress +CustomerEmail"?
replies(1): >>45040910 #
shadowgovt ◴[27 Aug 25 15:19 UTC] No.45040910[source]▶
Ideally, you automate a check like that. Because the answer turns out to actually be "humans are profoundly bad at that kind of pattern recognition."

A computer will be able to tell that the 497th has a misspelled `CusomerEmail` or that change 829 is a regexp failure that trimmed the boolean "CustomerEmailAddressed" to "CustomerEmailed" with 100% reliability; humans, not so much.

replies(3): >>45041140 #>>45041171 #>>45047557 #
const_cast ◴[28 Aug 25 02:10 UTC] No.45047557[source]▶
We already automate that, it's called a compiler. The human review is just for kicks for this type of thing.

Of course some languages... PHP... aren't so lucky. $customer->cusomerEmail? Good luck dealing with that critical in production, fuckheads!

replies(1): >>45051422 #
1. muglug ◴[28 Aug 25 12:38 UTC] No.45051422[source]▶
What if I told you there were tools for that
replies(1): >>45068742 #
2. const_cast ◴[29 Aug 25 20:01 UTC] No.45068742[source]▶
I'm aware of the tools, I write PHP daily.

The point is moreso that PHP won't stop you from doing that. It will run, and it will continue running, and then it will throw an error at some point. Maybe.

If the code is actually executed. If it's in a branch that's only executed like 1/1000 times... not good.