Re: [log4j-kotlin] Next release (`1.3.1`-vs-`1.4.0`)

2023-12-12 Thread Volkan Yazıcı
Thanks for the prompt action. 🙏 I will review the PR. Please keep in mind that you don't have sufficient rights to complete a release. A PMC member should volunteer as an RM (Release Manager), who, in your case, is Matt, I presume. That is, you can perform the git chores, but it is Matt who should

Re: Versioning scheme

2023-12-12 Thread Volkan Yazıcı
I agree with Ralph's view. It is not just the *view* in itself that I find appealing, but the (practically) deterministic nature of it. If one would closely look at Robert's and/or Piotr's explanation of patch-vs-minor strategy, it is pretty subjective, which is exactly the variable I am aiming to

Re: Versioning scheme

2023-12-12 Thread Piotr P. Karwasz
Hi Ralph, On Mon, 11 Dec 2023 at 22:43, Ralph Goers wrote: > > The rule for this seems extremely simple to me. The changelog uses > > > > >

Re: Versioning scheme

2023-12-12 Thread Volkan Yazıcı
On Tue, Dec 12, 2023 at 11:47 AM Piotr P. Karwasz wrote: > * we lack a way to classify dependency updates. A concrete example: > did the Commons DBCP2 bump to 2.11.0 change anything in > `log4j-jdbc-dbcp2`? I don't think so, the artifact compiles with > version 2.2.0 of the artifact. We are only

Re: Versioning scheme

2023-12-12 Thread Piotr P. Karwasz
Hi Volkan, On Tue, 12 Dec 2023 at 13:02, Volkan Yazıcı wrote: > > On Tue, Dec 12, 2023 at 11:47 AM Piotr P. Karwasz > wrote: > > > * we lack a way to classify dependency updates. A concrete example: > > did the Commons DBCP2 bump to 2.11.0 change anything in > > `log4j-jdbc-dbcp2`? I don't thin

Re: Versioning scheme

2023-12-12 Thread Christian Grobmeier
On Tue, Dec 12, 2023, at 09:43, Volkan Yazıcı wrote: > I agree with Ralph's view. It is not just the *view* in itself that I find > appealing, but the (practically) deterministic nature of it. If one would > closely look at Robert's and/or Piotr's explanation of patch-vs-minor > strategy, it is pre

Re: Versioning scheme

2023-12-12 Thread Ralph Goers
I have to agree with Piotr’s example. Simply upgrading a dependency doesn’t, on its own, change any code in Log4j. I see three possible solutions for this: 1. Do not allow any dependency updates until a “scheduled” mentor release (once every 2 or 3 months). Frankly I’d be fine with this except

Re: Versioning scheme

2023-12-12 Thread Ralph Goers
> On Dec 11, 2023, at 2:32 PM, Piotr P. Karwasz wrote: > > I propose to keep the old strategy: after a release we set the version > number to the next patch release. > Only if we merge a change that requires a minor bump (we can tag > those), we bump the release to the next minor version. Act

Re: Versioning scheme

2023-12-12 Thread Volkan Yazıcı
Piotr, could you give me a well-defined formula of your desired versioning scheme concerning minor/patch bumps? On Tue, 12 Dec 2023 at 15:19, Piotr P. Karwasz wrote: > Hi Volkan, > > On Tue, 12 Dec 2023 at 13:02, Volkan Yazıcı wrote: > > > > On Tue, Dec 12, 2023 at 11:47 AM Piotr P. Karwasz < >

Re: Versioning scheme

2023-12-12 Thread Volkan Yazıcı
Ralph, do you mean if all changes are a subset of bug fixes, deprecations, or updates, then it is a patch release? On Tue, 12 Dec 2023 at 16:23, Ralph Goers wrote: > I have to agree with Piotr’s example. Simply upgrading a dependency > doesn’t, on its own, change any code in Log4j. > > I see thr

Re: Versioning scheme

2023-12-12 Thread Ralph Goers
Yes. Ralph > On Dec 12, 2023, at 9:03 AM, Volkan Yazıcı wrote: > > Ralph, do you mean if all changes are a subset of bug fixes, deprecations, > or updates, then it is a patch release? > > On Tue, 12 Dec 2023 at 16:23, Ralph Goers > wrote: > >> I have to agree with Piotr’s example. Simply upg

Re: Versioning scheme

2023-12-12 Thread Piotr P. Karwasz
Hi Volkan, On Tue, 12 Dec 2023 at 17:00, Volkan Yazıcı wrote: > Piotr, could you give me a well-defined formula of your desired versioning > scheme concerning minor/patch bumps? I agree with what Ralph proposed. What I don't agree with is the current practice to set up the **next** release vers

Re: Versioning scheme

2023-12-12 Thread Volkan Yazıcı
Got it. I will land a log4j-changelog change accordingly, implement the CI magic, and update the release instructions. On Tue, 12 Dec 2023 at 17:41, Piotr P. Karwasz wrote: > Hi Volkan, > > On Tue, 12 Dec 2023 at 17:00, Volkan Yazıcı wrote: > > Piotr, could you give me a well-defined formula of

Re: Versioning scheme

2023-12-12 Thread Ralph Goers
Maybe you could add comments to the schema to define what the enumerations are intended for and what effect they have on the release? Ralph > On Dec 12, 2023, at 10:13 AM, Volkan Yazıcı wrote: > > Got it. I will land a log4j-changelog change accordingly, implement the CI > magic, and update th

Re: Versioning scheme

2023-12-12 Thread Volkan Yazıcı
Interesting. Could you share a concrete example, please? On Tue, Dec 12, 2023 at 8:16 PM Ralph Goers wrote: > Maybe you could add comments to the schema to define what the enumerations > are intended for and what effect they have on the release? > > Ralph > > > On Dec 12, 2023, at 10:13 AM, Volk