From 09e118d094ee8540cdd48723c964ad9f30ea0c73 Mon Sep 17 00:00:00 2001
From: Jerry Lee <oldratlee@gmail.com>
Date: Tue, 6 Aug 2024 11:41:36 +0800
Subject: [PATCH] chore: update `extra-enforcer-rules` maven plugin config, and
 improve profile config

---
 pom.xml | 82 ++++++++++++++++++++++++++++-----------------------------
 1 file changed, 40 insertions(+), 42 deletions(-)

diff --git a/pom.xml b/pom.xml
index 9d315385..6072f0dc 100644
--- a/pom.xml
+++ b/pom.xml
@@ -489,7 +489,8 @@
 			<id>default-logging-dependencies</id>
 			<activation>
 				<property>
-					<name>!enableLog4j2LoggingDependencies</name>
+					<name>enableLog4j2LoggingDependencies</name>
+					<value>!true</value>
 				</property>
 			</activation>
 			<dependencies>
@@ -516,14 +517,10 @@
 			<activation>
 				<property>
 					<name>enableLog4j2LoggingDependencies</name>
+					<value>true</value>
 				</property>
 			</activation>
 			<dependencies>
-				<dependency>
-					<groupId>org.slf4j</groupId>
-					<artifactId>slf4j-api</artifactId>
-					<optional>true</optional>
-				</dependency>
 				<dependency>
 					<groupId>org.apache.logging.log4j</groupId>
 					<artifactId>log4j-slf4j2-impl</artifactId>
@@ -536,42 +533,12 @@
 				</dependency>
 			</dependencies>
 		</profile>
-		<profile>
-			<id>enable-test-jar-if-not-maven-skip-test</id>
-			<activation>
-				<property>
-					<name>maven.test.skip</name>
-					<value>!true</value>
-				</property>
-			</activation>
-			<build>
-				<plugins>
-					<plugin>
-						<groupId>org.apache.maven.plugins</groupId>
-						<artifactId>maven-jar-plugin</artifactId>
-						<executions>
-							<execution>
-								<goals>
-									<!--
-										Guide to using attached tests
-										reuse the tests that you have created for one project in another
-											https://maven.apache.org/guides/mini/guide-attached-tests.html
-										How to create a jar containing test classes
-											https://maven.apache.org/plugins/maven-jar-plugin/examples/create-test-jar.html
-									-->
-									<goal>test-jar</goal>
-								</goals>
-							</execution>
-						</executions>
-					</plugin>
-				</plugins>
-			</build>
-		</profile>
 		<profile>
 			<id>enable-arch-unit-test</id>
 			<activation>
 				<property>
-					<name>!skipDefaultProfileArchUnitTest</name>
+					<name>skipDefaultProfileArchUnitTest</name>
+					<value>false</value>
 				</property>
 			</activation>
 			<dependencies>
@@ -617,6 +584,37 @@
 				</plugins>
 			</build>
 		</profile>
+		<profile>
+			<id>enable-test-jar-if-not-maven-skip-test</id>
+			<activation>
+				<property>
+					<name>maven.test.skip</name>
+					<value>!true</value>
+				</property>
+			</activation>
+			<build>
+				<plugins>
+					<plugin>
+						<groupId>org.apache.maven.plugins</groupId>
+						<artifactId>maven-jar-plugin</artifactId>
+						<executions>
+							<execution>
+								<goals>
+									<!--
+										Guide to using attached tests
+										reuse the tests that you have created for one project in another
+											https://maven.apache.org/guides/mini/guide-attached-tests.html
+										How to create a jar containing test classes
+											https://maven.apache.org/plugins/maven-jar-plugin/examples/create-test-jar.html
+									-->
+									<goal>test-jar</goal>
+								</goals>
+							</execution>
+						</executions>
+					</plugin>
+				</plugins>
+			</build>
+		</profile>
 		<profile>
 			<id>disable-javac-options-warning-for-java21+</id>
 			<activation>
@@ -658,12 +656,12 @@
 										<requireJavaVersion>
 											<version>21</version>
 										</requireJavaVersion>
-										<!-- https://www.mojohaus.org/extra-enforcer-rules/enforceBytecodeVersion.html -->
 										<enforceBytecodeVersion>
+											<!-- https://www.mojohaus.org/extra-enforcer-rules/enforceBytecodeVersion.html -->
 											<maxJdkVersion>${maven.compiler.source}</maxJdkVersion>
-											<ignoredScopes>
-												<ignoredScope>test</ignoredScope>
-											</ignoredScopes>
+											<excludes>
+												<exclude>com.tngtech.archunit:archunit</exclude>
+											</excludes>
 										</enforceBytecodeVersion>
 									</rules>
 								</configuration>