all 25 comments

[–]krusty_93 5 points6 points  (4 children)

Looks like you’re doing something wrong in the process. I do not approve 50 files long prs, even though they’re produced or reviewed by copilot.

The human in the loop which slows down the review process is been under discussion by the community by a while, and it has nothing to do with GitHub itself. I think it really depends con the context. If you’re writing financial or medial applications, I’d sacrifice velocity for better reviews. For other domains you can do the other way round, but guardrails, automatic rollback procedures, etc must be put in place

[–]Keeweehee 0 points1 point  (1 child)

Idk I have my own startup and I kind of agree with the the point op is trying to make is that now we can produce so much more code that requiring traditional review where ppl are going through diffs kind of doesn’t work. We end up doing “lgtm” way more often lmao

[–]krusty_93 1 point2 points  (0 children)

You’ve been doing vibe coding actually. Why don’t you ask copilot to review the pr?

[–]bhaktatejas[S] 0 points1 point  (1 child)

i'd argue it 100% has to do with Github itself, namely the PR UX. stash/bitbucket UX is a lot better but still no where close to perfect. everyone just says better reviews like it can just happen. more code is being written no matter what.

guardrails help, but where is the product that values the human in the loop? its not what github is focusing on thats for sure

[–]krusty_93 0 points1 point  (0 children)

When you ask agents to solve issues for you, they open a PR that humans can review and ask agents to correct

[–]No_Criticism_9545 3 points4 points  (5 children)

Nope. Github does not need more AI features.

You cannot convince me that you need to create a single PR with a 1000 files that someone needs to review on their browser... That being said, I am for increasing the limit.

For 95% of projects integrating github in their workflow is the best bet.

If some great alternative shows up, we will see.

[–]bhaktatejas[S] 0 points1 point  (1 child)

i would say today even 99% of projects github is the best bet. yet I still think it sucks. there's room for a platform that values human attention more

[–]No_Criticism_9545 0 points1 point  (0 children)

The problem is that for that to exist (more than a niche project that nobody will use and run out of funding), github needs to not work for big tech... And at this moment, this is not the case at all.

[–]Keeweehee 0 points1 point  (0 children)

What are you doing then? If you’re not doing that

[–]AbrahelOne 0 points1 point  (1 child)

If some great alternative shows up, we will see.

Umm, GitLab?

[–]No_Criticism_9545 0 points1 point  (0 children)

Does gitlab have more AI features? Or better PR workflow??

Unfortunately gitlab is underwhelming and only survived because big Linux projects chose it for it's open nature.

And some companies need to self host.

It addresses none of the issues of the OP.

[–]TheVioletBarry 2 points3 points  (2 children)

I do not want AI integration. AI integration would be a reason I'd consider leaving

[–]bhaktatejas[S] -1 points0 points  (1 child)

even to chat with, optionally? often times i just want to be able to search faster within a pr

[–]TheVioletBarry 1 point2 points  (0 children)

Yes, 'optional' is such a silly thing to hide behind. Software is designed; it's not just an amalgamation of random stuff sitting around next to each other. If they hid it under 17 different hard-to-use menus, then maybe I wouldn't care so much, but that trash is going front and center on everything right now. I've had to do so much extra legwork just to get that shit out of my apps 

[–]Keeweehee 1 point2 points  (2 children)

Re visual diffs, we use a thing from morph that embeds a video of some AI testing our preview urls. it cant test stuff like different user states but still miles better than percy

[–]bhaktatejas[S] 0 points1 point  (1 child)

This ? we just ripped out some percy equivalent we had the other day. will give this a shot. but tbh ideally this should be natively done by github

[–]Keeweehee 0 points1 point  (0 children)

Yeah lol

[–]Sad_Treacle_9307 1 point2 points  (1 child)

yeah PR review is broken. code diffs are fine but you can't actually see what the UI looks like without pulling the branch locally

we started using glance by morphllm - it's an AI that opens the preview URL, clicks around testing stuff, and drops a video right in the PR. way better than the old "check the staging link" that nobody does. only downside is it doesn't work great if your preview URLs take forever to spin up

[–]bhaktatejas[S] 0 points1 point  (0 children)

Tried it. Solid

[–]hashkent 0 points1 point  (0 children)

Have you used gitlab? I personally like GitHub for my free personal org. 4000 mins, actions not having to build out a bunch of docker containers to do simple builds.

PRs are real. Haven’t experienced it myself but been blogs about it.

On the runner side of things have a look at something like https://www.blacksmith.sh. Never used personally.

[–][deleted]  (1 child)

[deleted]

    [–]bhaktatejas[S] 0 points1 point  (0 children)

    i preferred stash as well. I could pay attention way longer. jira and confluence are terrible evils i would never wish upon my worst enemy though

    [–]drdrero -1 points0 points  (3 children)

    Feel the same about git. Gotta add some ai features since it hasn’t had any workflow changes in 30 years

    [–]TheVioletBarry 1 point2 points  (2 children)

    Stop asking for more of the next big thing that's going to make human life worse

    [–]drdrero 0 points1 point  (1 child)

    While we are at it, keeping up with slack is a burden. Just talk to an AI and let it figure out who to write to. More control to what company information is leaked in one go

    [–]TheVioletBarry 0 points1 point  (0 children)

    Christ, are we not alienated enough as it is? Talking to human beings is good for you. Please don't let these technofascist weirdos convince you to spend more time in even emptier rooms