3 ms·
Why require PR approval if people can do it themselves? Then it just becomes useless ceremony over allowing people to merge without approvals or PRs
by dtech 8y ago
Why require PR approval if people can do it themselves? Then it just becomes useless ceremony over allowing people to merge without approvals or PRs
- wrs 8y agoPRs are a useful artifact that can trigger CI builds, provide an organized way to discuss changes, record reasoning behind multi-commit changes, etc. etc.
- Raphmedia 8y agoBecause that way people "stamp" their work. There's no excuse, you can't lie to yourself about half done tasks. You made a PR and even approved it.