Hacker Newsnew | past | comments | ask | show | jobs | submit | cbovis's commentslogin

AFAICT this is already baked into the GitHub Copilot agent. I read its sessions pretty often and reviewing/testing after writing code is a standard part of its workflow almost every time. It's kind of wild seeing how diligent it is even with the most trivial of changes.


GH Copilot is definitely far better than just a linter. I don't have examples to hand but one thing that's stood out to me is its use of context outside the changes in the diff. It'll pull in context that typically isn't visible in the PR itself, the sort of things that only someone experienced in the code base with good recall would connect the dots on (e.g. this doesn't conform to typical patterns, or a version of this is already encapsulated in reusable code, or there's an existing constant that could be used here instead of the hardcoded value you have).


I've also noticed this explosion of code review tools and felt that there's some misplaced focus going on for companies.

Two that stood out to me are Sentry and Vercel. Both have released code review tools recently and both feel misplaced. I can definitely see why they thought they could expand with that type of product offering but I just don't see a benefit over their competition. We have GH copilot natively available on all our PRs, it does a great job, integrates very well with the PR comment system, and is cheap (free with our current usage patterns). GH and other source control services are well placed to have first-class code review functionality baked into their PR tooling.

It's not really clear to me what Sentry/Vercel are offering beyond what copilot does and in my brief testing of them didn't see noticeable difference in quality or DX. Feels like they're fighting an uphill battle from day one with the product choice and are ultimately limited on DX by how deeply GH and other source control service allow them to integrate.

What I would love to see from Vercel, which they feel very well placed to offer, is AI powered QA. They already control the preview environments being deployed to for each PR, they have a feedback system in place with their Vercel toolbar comments, so they "just" need to tie those together with an agentic QA system. A much loftier goal of course but a differentiator and something I'm sure a lot of teams would pay top dollar for if it works well.


Looks to be affecting our pipelines that rely on Playwright as they download images from Azure e.g. https://playwright.azureedge.net/builds/chromium/1124/chromi... which aren't currently resolving.


No tip


Suggestion that the admin is vibe governing: https://bsky.app/profile/amyhoy.bsky.social/post/3lluo7jmsss...




> I've now switched my primary HN link from the home page to /active, where, thankfully, flagged posts still show up.

Great tip, thanks!


Yea, the difference between home and /active has never been starker. I feel like HN's well-intentioned user-flagging system assumed good faith, and was not built to handle the obvious partisan brigading that we've been seeing.


> system assumed good faith

Ironically also the reason that many checks and balances in the US government seem to be failing.


> Why isn't it front-page news on HN rather than being flagged? Is it just because Musk's "DOGE" is in the title?

Essentially yes. Certain users are flagging because they don't want any discussion related to DOGE on HN. See for example https://news.ycombinator.com/item?id=43036254.


LOL... wow! I thought the "national cyberattack" and the root override/hijacking was far more relevant than the connection with the organization which shall not be named which is nominally led by he who shall not be named.


Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: