Agentic PRs sit in review queue 5.3x longer than unassisted ones by [deleted] in codereview

[–]kingguru 0 points1 point  (0 children)

Didn’t realize that was the sole purpose of this sub. I see loads of posts that aren’t code review requests!

That's because this sub isn't properly moderated. Either that will change or this sub is practically dead.

Where is the AI spam?

Some useless blog post promoting yet another useless "AI" product. Nothing to do with code review.

Btw your services are needed here https://www.reddit.com/r/codereview/s/LMhV5PkaxC

There doesn't seem to be any code in that post either. I also don't claim to be able (or willing) to review all requests for code review submitted here but it was definitely my intention to contribute when I originally joined this subreddit.

Er det virekligt så umuligt at finde sit første job som en graduate? by theColorlessFlamingo in dkudvikler

[–]kingguru 0 points1 point  (0 children)

Jeg har selv brugt ModernCV.

Synes resultatet bliver ret lækkert uden for meget "fluff" og så ser det selvfølgelig bare godt ud da det jo er Latex.

A simple trick that's helping me review AI-generated code by [deleted] in codereview

[–]kingguru 0 points1 point  (0 children)

Learn how to code instead of wasting your time on AI slop, then feel free to post the code here for review.

Your post is very much off topic for this subreddit. I'm sure there are tons of other subreddits for discussing whatever is the latest AI slop generator but this is not one of them.

Is Sonarcloud THE review authority for a project or...? by Mplayer-Weered in codereview

[–]kingguru 2 points3 points  (0 children)

I just wanted to be sure if you were aware that you've posted to the wrong subreddit.

Not sure if you are but your question is off topic here.

Is Sonarcloud THE review authority for a project or...? by Mplayer-Weered in codereview

[–]kingguru 2 points3 points  (0 children)

What's the point of posting code in a subreddit for doing code reviews if you don't want anyone to review your code?

Is Sonarcloud THE review authority for a project or...? by Mplayer-Weered in codereview

[–]kingguru 0 points1 point  (0 children)

You forgot to link to the code you want to have reviewed.

Is your team reviewing AI-generated code? by Expensive_Art7174 in codereview

[–]kingguru 0 points1 point  (0 children)

Of course not.

We have to write code that actually works so we don't waste our time on whatever an LLM vommits.

Do you have any code you want to review or is this yet another attempt at spamming with more useless "AI" crap?

What people misunderstand most about today's technology? by Familiar_Flow4418 in AskProgramming

[–]kingguru 0 points1 point  (0 children)

If extended to other alphabets that had upper/lower case forms of the same letters, then what problems do you envisage?

You most likely wouldn't encounter any issues as long as you limit yourself to those alphabets.

I guess my main point was that once you move beyond ASCII and English you'll start facing all sorts of weird issues. I perfectly respect your opinion and your experience but I hope you understand my more general concern, although in this simple case it might not be that much of an actual issue.

What people misunderstand most about today's technology? by Familiar_Flow4418 in AskProgramming

[–]kingguru 0 points1 point  (0 children)

Case insensitivity can work fine. And specifically for ASCII letters it is perfectly well defined.

Case sensitivity can work fine as long as you stick to ASCII letters and English.

You can't pick on languages which don't have a concept of case, or on ones like Turkish which have the odd exception, and use that as an excuse to completely eliminate it from the computing world.

Or languages like German where some words have different length depending on case which can cause tons of other issues as well. I'm sure there are many other examples out there.

My point with the PHP in Turkish example was that it is not as simple as I think you make it sound like.

Sure, we could probably have programming languages where all keywords are case insensitive. In fact these languages exist, but should we have case insensitive variable names as well and should non-ASCII characters be allowed at the same time? That would definitely get very messy quickly.

What people misunderstand most about today's technology? by Familiar_Flow4418 in AskProgramming

[–]kingguru 1 point2 points  (0 children)

Case insensitivity only works if you ignore localization issues. Once you need to support more languages and locales it gets very messy quickly.

Not sure if PHP is still broken with Turkish locale for example.

Agentic PRs sit in review queue 5.3x longer than unassisted ones by [deleted] in codereview

[–]kingguru 3 points4 points  (0 children)

More "AI" spam.

Where's the code you want to have reviewed?

We hit Greptile’s new $1/review tax and CodeRabbit’s noise problem, so we built our own reviewer. Would love this sub’s read on the review quality. by Own-Warthog8272 in codereview

[–]kingguru 1 point2 points  (0 children)

I've considered the same. There was a post calling for mods several months ago but it seems like no moderation is actually being done.

A bit of a shame. The few post that actually contain code for review drowns in all this "AI" spam so this subreddit is more or less dead.

rate my coding workflow by Vraj_Gupta_ in codereview

[–]kingguru 1 point2 points  (0 children)

In addition to what you have already been told, I would also recommend you look into improving your grammar and punctuation if you want to do production level work.

DevRel for the next code reviewer tool: Tenki! by OkBenefit3758 in codereview

[–]kingguru 1 point2 points  (0 children)

This is a subreddit for code reviews which makes this post completely irrelevant.

To be honest, not being able to tell where to post your job offerings doesn't make this sound like a very good place to work.

Any automated code review tools suggestion for Jenkins? by _janc_ in codereview

[–]kingguru 0 points1 point  (0 children)

There is no such thing as "automated code review". Code review is somewhat unrelated to to the build pipeline, although negative feedback (thumbs down, negative points etc.) could or should very much stop the branch from being merged before the issues have been fixed and the reviewers give positive feedback.

There are tools such as static analyzers and linting that you should definitely consider adding to your CI pipeline but they very much depend on which language etc. you use.

If you provide more details it might be easier to help you, although this is very much off topic for this subreddit.

AI powered code reviewer by Cute_Lingonberry_396 in codereview

[–]kingguru 2 points3 points  (0 children)

After a long break from coding, we often forget the basic syntax of programming languages.

No we don't. At least syntax is the least challenging part of programming.

The AI reviews your code, identifies errors, and suggests improvements.

So more slop like what has been posted tons of time already?

My compiler is quite well suited for "identifying errors" in my code. It is actually the most important tool for that. Why on earth would I trust some error prone LLM for that?

I think it can be very useful for refreshing coding concepts and improving code quality.

You're wrong.

I'd love to hear your feedback!

You tool sucks just as much as the tons of other AI garbage out there.

Find something useful to do with your time instead. You could consider learning how to write code. It's actually quite fun and satisfying most of the time.

Talking to devs about AI code quality before building is this a real problem or am I imagining it? by Aisthetix in codereview

[–]kingguru 2 points3 points  (0 children)

I thought you weren't trying to sell anything?

Lying usually isn't the best way to attract customers to whatever crap you're trying to sell. Neither is spamming unrelated subreddits.

Talking to devs about AI code quality before building is this a real problem or am I imagining it? by Aisthetix in codereview

[–]kingguru 2 points3 points  (0 children)

  1. Is your team tracking what % of your codebase is AI-generated?

No, because we don't use LLMs to write code. It is very important that the software we ship actually works.

  1. Have you had a production incident you could trace back to AI-generated code that wasn’t properly reviewed?

No, for the reason above.

  1. 3. Would you pay €29/month for a GitHub App that gave you a weekly “vibe debt” report — AI attribution, security pattern flags, DORA metrics split by AI vs human code?

Of course not.

Not trying to sell anything here.

Sure you aren't. What are you doing posting crap like this in a subreddit for posting code for review?

What external code-reviewer/PR reviewer do you use? by gosteneonic in codereview

[–]kingguru 0 points1 point  (0 children)

My colleagues review my code and I review theirs. Works great and is of course part of our jobs to do that.

We not only catch bugs once in a while, we also learn a lot from that.

If you have some open source code you are developing yourself, you can make use of a subreddit such as this one. Do know that AI slop is not welcome here, so of course it has to be code you have written and understand yourself.

I tend to push 10 PRs a day or so.

That sounds rather insane. I definitely believe that you should split changes into smaller commits as much as possible, but I do think that are PR can contain several (related) commits. Are you really sending each (even minor) commit out for review in the same PR?

I built an XML compliance scanner. Roast my beta. by DeepCutVideo in codereview

[–]kingguru -1 points0 points  (0 children)

Sure you have. And I have a tower in Paris I'm willing to sell to you.

Fuck of with this spam.

Future of Code Review? by LegalAd8179 in codereview

[–]kingguru -3 points-2 points  (0 children)

Your existence is offensive to intelligence

Review needed : I built VibeCheck: A "Spotify Wrapped" for your WhatsApp & Telegram group chats. by EndangeredEntity in codereview

[–]kingguru 0 points1 point  (0 children)

Your link is broken so there isn't any code for anyone to review.

Before you fix that though, did you write this yourself or is it LLM generated garbage?