Hi Bertil, I know that I am not reviewing much these days so my voice probably doesn't count much as I am a bit of a bottleneck potentially, but I am in principle against a no review policy, even for trivial stuff.
The main reason being that there should be always at least 2 people aware of some changes. The author and any reviewer in this case. Moreover, it is easy to make stupid errors when doing "easy" things... Cheers On Mon, Nov 14, 2022, 21:27 Bertil Chapuis <bchap...@gmail.com> wrote: > Hi Everyone, > > The current policy is to have at least one review before merging. Some > changes need to be reviewed. However, I believe that many incremental > changes (config, cleaning, typos, etc.) can be reviewed after being merged. > I'd like committers to feel empowered to modify and improve the code base > when needed. Do you think we should relax the current policy and disable > the review requirement? > > Best, > > Bertil >
