Hi all,

Inspired by the way the Log4Net team works, I would like to propose a
switch from our CTR policy to RTC:

* every change to the main branches should go through a pull request,
* pull requests can be merged if they have 1 positive review and no
requested changes. To allow everyone to look at the PR, let's not
merge them before at least 24 hours have passed.
* if a pull request does not receive any review in 72 hours, as a last
resort we merge them without a review.

I would like to apply this policy to our most active repos:

* l-log4j2
* l-log4j-kotlin
* l-log4j-scala
* l-log4j-transform
* l-log4j-tools
* l-log4net

I am NOT proposing this change to the equally active l-log4cxx
repository, since the Log4cxx team has an established workflow that
does not use formal reviews.

I would prefer to explicitly add a branch protection rule that
requires reviews. If a PR is not reviewed within 72 hours, please ping
the Slack channel so someone can review it.

What do you think?

Piotr

Reply via email to