Re: LOG4J2-3228 - Remove support for Serializable

2023-01-14 Thread Matt Sicker
So I’ve updated the PR to remove the generic parameter from Layout. I’ve also found the remaining classes implementing Serializable and updated them which includes several interesting things like: * Logger (all of them) * Level * Message * MessageFactory * LogEvent * ThreadContextMap * ThreadCon

Re: Import order and POM formatting

2023-01-14 Thread Matt Sicker
I’m ok with whatever ordering we use. As much as spotless build failures annoy me, I am loving the increased consistency in some of the annoying parts of code style. — Matt Sicker > On Jan 14, 2023, at 03:52, Piotr P. Karwasz wrote: > > Hi all, > > I noticed a difference in the spotless confi

Import order and POM formatting

2023-01-14 Thread Piotr P. Karwasz
Hi all, I noticed a difference in the spotless configuration of our repos. The `logging-log4j2` repo uses: java,org,com,\# The `logging-log4j-tools` repo uses: java,javax,org.apache.logging,,\#java,\#javax,\#org.apache.logging,\# I copied the latter to `logging-log4j-transform`, so I don't di