6 ms·
That remains to be seen. The whole concept is still in its infancy. An AI reviewer should have caught these issues when they were PRs. But you see, it wasn't e
by vga1 17d ago
That remains to be seen. The whole concept is still in its infancy. An AI reviewer should have caught these issues when they were PRs.
But you see, it wasn't even a PR. It was just DHH making a straight up commit on the main branch (as far as I can see). With a message "Do all the additional Docker configuration needed". Was it even AI-assisted? At least the commit message wasn't, AIs write smarter commit messages than that.
I said on my earlier comment that DHH and AI get ridiculed automatically because people hate both. That doesn't mean that the ridicule isn't always justified.
https://github.com/omacom/omarchy/commit/25799ee91f54c35e6d340df3aae8ac2b21fae0a4 https://github.com/omacom/omarchy/commit/25799ee91f54c35e6d3...
- dngray 17d agoWhich tells me that they don't really take security seriously because everyone knows exposing the docker socket is dangerous. I would almost bet that AI would warn about that.
- vga1 17d ago> I would almost bet that AI would warn about that. Exactly. If they had a review process. Which they didn't, at least for that change. https://xkcd.com/1200/ https://xkcd.com/1200/