Hi, Bertil Chapuis <bchap...@gmail.com> wrote: > ...Do you think we should relax the current policy and disable the review > requirement?...
I think it's good for the project to define whether it wants to operate in CTR or RTC mode (Commit-Then-Review or Review-Then-Commit, [1]) IMO declaring CTR generally (which means no pre-merge review required) and if needed RTC for some critical parts of the code works well. People are supposed to watch the commit logs and especially review changes before making releases, so in general I think CTR is fine. -Bertrand [1] https://www.apache.org/foundation/glossary.html --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@baremaps.apache.org For additional commands, e-mail: dev-h...@baremaps.apache.org
