3 ms·
My record is 45 comments on a single review. Merge conditions were configured so that every comment must be resolved. If PR author can satisfy it - I'm fine wi
by ivanjermakov 1y ago
My record is 45 comments on a single review. Merge conditions were configured so that every comment must be resolved.
If PR author can satisfy it - I'm fine with it.
- cryptonym 1y agoThey will let AI somewhat satisfying it and ask you for further review
- ivanjermakov 1y agoReminds me of curl problems with vulnerability report: https://news.ycombinator.com/item?id=43907376 https://news.ycombinator.com/item?id=43907376 At that point it is just malicious.
- cryptonym 1y agoSome people genuinely believe agentic coding works great and they mastered it. Someone who PR a simple feature with its own DSL probably is on that team and won't see the issue with their way. They may think you are too old and resist AI. They probably would tell you if that's too much for your old fashioned coding skills, then just use an agent for the PR. If you think that way, who cares about the code and additional DSL? If there is an issue or evolution required, we'll let AI work on it. If it works, just let it merge. Much cheaper than human reviewing everything. I hate it, maybe I'm too old.