Top
Best
New

Posted by xiaoyu2006 17 hours ago

GitHub New pull request overview page in public preview(github.blog)
168 points | 114 commentspage 3
varun_chopra 16 hours ago|
> The merge box, distilled

Ugh

rafram 15 hours ago||
Shovel manufacturer responds to complaints about customers throwing piles of manure in neighbors’ yards: “We’re trialing a free window cleaning service to help you get a clearer view of the manure”
kccqzy 15 hours ago||
> As people need to review more and more pull requests, time spent scrolling adds up

I can’t believe people would write this as the justification. You can press PgDn on your keyboard a few times or just End once to go to the bottom. If you are on a MacBook and your keyboard doesn’t have these keys, press Fn-Down instead of PgDn and Fn-Right instead of End. When done, use the opposite keys (PgUp/Home/Fn-Up/Fn-Left). I have never once found that scrolling takes too much time.

verdverm 12 hours ago|
It seems the consensus isn't humans did not write it
conceptme 6 hours ago||
95% if not more if claude thinks it's ok it's going in the sheer volume of code changes we are now doing are simply not doable by hand anymore.

It's basically the wild wet now :D bots reviewing bots.

Ideally only risky merges should get human attention, and some guard rails to enforce it but mistakes have been made.

ivanjermakov 15 hours ago||
Finally approve button is available on the main screen without having to open diff view which takes ages to load. Still two clicks though.
TingPing 13 hours ago|
You didn’t have to open the diff before.
hoektoe 6 hours ago||
So we getting UI updates just as humans are looking less and less at these screens via github....
stock_toaster 15 hours ago||
Just more slopstick on the pig. :/
d33 3 hours ago||
I know it's boring at this point, but seeing Claudish headers like "The merge box, distilled" and "Keep your place. Focus your attention" just screams SLOP at me.

I can't read it. It's not really text.

smithcoin 11 hours ago||
too little too late - we migrated to self hosted gitea.
cedws 16 hours ago|
Can you fix the inbox first please before reinventing more UI? The inbox is utterly broken and slow. It frequently says my PRs are waiting for review when they’ve already been approved.
More comments...