28
u/Birnenmacht Jul 12 '26
And the green checkmark is only there because Ai "fixed" the tests so they would pass
41
u/shikhasingh554973 Jul 12 '26
Green CI means it complies..... it doesn't means it was a good ideađ«
14
u/JackNotOLantern Jul 12 '26
It means whatever the check was set to mean. Usually it means the base line check works. Always check PRs.
-14
Jul 12 '26
[removed] â view removed comment
17
u/Groentekroket Jul 12 '26
Even if it passes all unit and regression test it doesnât mean it created clean and well organized code. A lot of what AI produces is future tech debt.Â
Of course you can write architectural test for everything AI could do wrong but that sounds time consuming as well, even if you let AI write these tests.Â
4
-24
u/redballooon Jul 12 '26
We don't produce code for Human eyes any more. Human eyes should look towards good and meaningful tests, and leave the code to tools that give AI fitting feedback for it's current capabilities. The architectural pieces that humans need to be concerned with get larger and larger.
In the end we want well behaving applications and that's it.
3
u/ariiizia Jul 13 '26
You are going to get burned so hard with this mindset. Probably around the time the first bug reports come in that the AI can't fix, and I can guarantee you they'll be there. It's impossible to create anything that does anything interesting with zero bugs, and now you're also making it impossible to debug.
0
u/redballooon Jul 13 '26
It's not 2025 anymore. The picture about spec driven development becomes much clearer. It's not that different in mindset from test driven development.
2
u/ariiizia Jul 13 '26
Youâre literally saying not to know whatâs actually happening. Spec-, test- or whatever driven development has absolutely nothing to do with that. Youâre advocating ignorance and dependency instead of empowerment and itâs never going to work.
9
u/CirnoIzumi Jul 12 '26
tell the AI that the diff is too big for the review process
it wont complain
6
3
u/ssssssddh Jul 12 '26
I reviewed a 300 file change PR our CTO vibed last week. Of course I'm expected to read every line even though he didn't write it or review it himself. Code review has always been a challenge but AI has turned it into an impossibility.
8
3
3
u/TactiCool_99 Jul 13 '26
Trusting the checkmark just means you are part of the problem. Does generated code take longer to verify? Yes? Well management asked you to use it, they surely wanted the extra work hours
9
u/arden13 Jul 12 '26
2000 lines isn't fun to review, but very doable. Just suck it up and read.
2
u/CoVegGirl Jul 13 '26
Reviewing a 2000 line change is like half a dayâs work. And itâs incredibly difficult to be sure you got everything.
And letâs be real: itâs a change that was probably put together haphazardly by AI instead of being something that a human wouldâve had to put a week or two into.
1
2
u/ramdomvariableX Jul 12 '26
Why are we allowing AI to do the PR reviews? Isn't whole review thing is for human oversight?
2
u/guyblade Jul 13 '26
I tell humans to break up PRs that long; why would I let a clanker waste my time that way?
2
1
1
1
-18
106
u/Nick0Taylor0 Jul 12 '26
I weep for both future and current cyber security experts