Hi Volkan, On Wed, 18 Sept 2024 at 09:46, Volkan Yazıcı <vol...@yazi.ci.invalid> wrote: > Changes can be exempt from this policy if and only if they suffice *only > one* of the following conditions: > > 1. Grammatical corrections to the documentation (incl. Javadoc and > release notes) > 2. Code typo fixes¹ less than 10 LoC > 3. Infrastructure fixes¹ touching to `pom.xml` or CI scripts, and less > than 10 LoC > > ¹ A "fix" is assumed to not introduce a change in the expected behaviour of > the associated component. Changing the expected behaviour does not qualify > a fix.
For Log4j I would also prefer 1 and 2 to go through a PR. It will be easier for me to nag the author to port his changes to `main` this way. The port to `main` of a `2.x` PR probably does not require a PR. Regarding 3, sure, debugging some GitHub actions problem is impossible without pushing it to a branch. However, we don't necessarily need to use `2.x` or `main` for those tests. Piotr