Hacker News new | ask | show | jobs
by cratermoon 942 days ago
> PRs work best when it’s mostly a sanity check and rubber stamp for those you trust on small changes.

I'm 99.999% sure that kind of rubber stamp check can be automated. Then why have PRs for that at all?

1 comments

Record keeping for the most part. You need something to attach the CI jobs to and PRs are a good choice. There are plenty of cases were a deep review should be preformed. Always presenting that opportunity is good instead of having to do something different from the submitters point of view.
CI jobs trigger on the change, not the PR creation, so there's no need for the extra step. There's no need for a PR for trivial stuff. In the context of teams "internal to a company who know and trust each other", when deep review is still important, is the PR process useful, or is it busywork? Why not have the deep review happen during the development, rather than after?