6 pointsby tomerbd3 hours ago1 comment
  • 4lx8718 minutes ago
    Unpopular opinion: PRs are a massive waste of everyone’s time.

    Competent engineers committing working increments of software straight to main and continuously integrated and deployed > PRs of any sort.

    • kypro6 minutes ago
      This is an interesting take. Especially these days when AI reviews are fairly decent and arguably comparable to, if not better than, most human reviews in most ways.

      I did this on a small microservice for a project recently where risk was constrained and I didn't see much value or appetite for human reviews. AI review, if clear, merge straight to main.

      I suppose the risk is you have a vibe coder with AI psychosis blasting stuff into main on a mature project? Do you have have any sense for level of seniority, team size and size of code base that would allow this?

      I get the sense everyone would need to be very senior 10+ years experience, with a relatively small team (< 10), on a medium sized and relatively modern code base?