Remix.run Logo
fallingbananna 2 hours ago

That's exactly how I treat reviewing AI generated PRs at work.

Except... the next day they are still there and I have to push myself a bit further through the wall of changes before closing the tab again.

RobKohr 2 hours ago | parent [-]

If it takes more than 20 min to review a pr, ask the dev to come on a call and explain what the code does.

pc86 an hour ago | parent [-]

It must be nice to work on such small-scale systems that you can reasonably expect PRs to be this quick.

zamadatix an hour ago | parent | next [-]

I don't know it has as much to do with scale of the system vs the general architecture. E.g. the system I primarily work with these days has millions of lines but most PRs are for a small changes which are well contained in scope by the overall architecture.

8note an hour ago | parent | prev | next [-]

reasonably you should already know what the code and project is supposed to do before they even start writing code, so you can get directional feedback in.

then you are maybe reviewing 1 out of 7 PRs that implement the agreed upon change

gedy an hour ago | parent | prev [-]

It's a cultural thing, but you can do incremental PRs towards a large goal. Giant PRs that are expected to be reviewed never really seemed worth it imho.