Comment by gus_massa
8 years ago
In many small project the owner (o small set of owners) just commit the changes without approval. In same case on person writes more than the 50% of the commits, and it's not practical to get someone to review the code.
In this case abe33 has the 75% of the commits, someone else 15% and the rest is a bunch of people with 1% or less.
Once your project gets to a certain level of users or activity, you should still be submitting PRs or MRs for comment before merger.
With our server toolkit in a project I work on, we have 2 devs and 5 active users, with the devs being 2 of those, but we still manage to at least put every change in a PR, with a minimum review and comment time of 24 hours unless it's a security issue or major bug fix.
It's not hard, and it makes you actually justify your change and have talented second eyes point out minor bugs or edge cases to you.
Direct commits are only used for version bumps for the auto build/release thingy.
Your method may be better, but there are a lot of small and medium projects in the wild that don't follow it.
They should.