Comment by MrStonedOne
8 years ago
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.