Cleanup project and fix version plugin dependency (#29)

PlaytikaOSS · Oct 26, 2023 · 6189722 · 6189722
1 parent dc2080b
commit 6189722
Show file tree

Hide file tree

Showing 6 changed files with 30 additions and 67 deletions.
diff --git a/examples/mixin-maven-plugin/mixin-example/pom.xml b/examples/mixin-maven-plugin/mixin-example/pom.xml
@@ -7,7 +7,7 @@
     <name>${project.groupId}:${project.artifactId}</name>
 
     <properties>
-        <version.com.playtika.maven.plugins>1.0.0-SNAPSHOT</version.com.playtika.maven.plugins>
+        <version.com.playtika.maven.plugins>1.0.1</version.com.playtika.maven.plugins>
     </properties>
     <modules>
         <module>mixin1</module>

diff --git a/mixin-maven-plugin/pom.xml b/mixin-maven-plugin/pom.xml
@@ -20,13 +20,11 @@
         <dependency>
             <groupId>org.apache.maven</groupId>
             <artifactId>maven-core</artifactId>
-            <version>${maven-core.version}</version>
             <scope>provided</scope>
         </dependency>
         <dependency>
             <groupId>org.codehaus.plexus</groupId>
             <artifactId>plexus-utils</artifactId>
-            <version>${plexus-utils.version}</version>
         </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
@@ -50,7 +48,6 @@
                 <plugin>
                     <groupId>org.codehaus.plexus</groupId>
                     <artifactId>plexus-component-metadata</artifactId>
-                    <version>${plexus-component-metadata.version}</version>
                     <executions>
                         <execution>
                             <id>generate-metadata</id>
@@ -89,25 +86,6 @@
                 <groupId>org.codehaus.plexus</groupId>
                 <artifactId>plexus-component-metadata</artifactId>
             </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-enforcer-plugin</artifactId>
-                <executions>
-                    <execution>
-                        <id>enforce-maven</id>
-                        <goals>
-                            <goal>enforce</goal>
-                        </goals>
-                        <configuration>
-                            <rules>
-                                <requireMavenVersion>
-                                    <version>3.0</version>
-                                </requireMavenVersion>
-                            </rules>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
         </plugins>
     </build>
 </project>
diff --git a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/ArtifactFetcher.java b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/ArtifactFetcher.java
@@ -5,7 +5,7 @@
 import org.apache.maven.execution.MavenSession;
 
 public interface ArtifactFetcher {
-    public Artifact createArtifact(String groupId, String artifactId, String type, String classifier, String version);
+    Artifact createArtifact(String groupId, String artifactId, String type, String classifier, String version);
 
-    public void resolve(Artifact artifact, MavenSession mavenSession) throws ArtifactResolutionException;
+    void resolve(Artifact artifact, MavenSession mavenSession) throws ArtifactResolutionException;
 }
diff --git a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/FakeMixinMojo.java b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/FakeMixinMojo.java
diff --git a/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/MixinModelMerger.java b/mixin-maven-plugin/src/main/java/com/playtika/maven/plugins/MixinModelMerger.java
@@ -127,8 +127,8 @@ public void mergeDistributionManagement(Model target, Model source) {
     /**
      * Fully merges pluginContainers with their plugins, their executions and their configuration
      *
-     * @param targetPlugin
-     * @param sourcePlugin
+     * @param targetContainer
+     * @param sourceContainer
      * @param context
      */
     private void mergePluginContainers(PluginContainer targetContainer, PluginContainer sourceContainer, Map<Object, Object> context, boolean addTargetPlugin) {

diff --git a/pom.xml b/pom.xml
@@ -136,20 +136,9 @@
 
     <build>
         <plugins>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-compiler-plugin</artifactId>
-                <version>${maven-compiler-plugin.version}</version>
-            </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-surefire-plugin</artifactId>
-                <version>${maven-surefire-plugin.version}</version>
-            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-source-plugin</artifactId>
-                <version>${maven-source-plugin.version}</version>
                 <executions>
                     <execution>
                         <id>attach-sources</id>
@@ -162,7 +151,6 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-javadoc-plugin</artifactId>
-                <version>${maven-javadoc-plugin.version}</version>
                 <executions>
                     <execution>
                         <id>attach-sources</id>
@@ -204,6 +192,31 @@
 
         <pluginManagement>
             <plugins>
+                <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
+                    <artifactId>maven-compiler-plugin</artifactId>
+                    <version>${maven-compiler-plugin.version}</version>
+                </plugin>
+                <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
+                    <artifactId>maven-surefire-plugin</artifactId>
+                    <version>${maven-surefire-plugin.version}</version>
+                </plugin>
+                <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
+                    <artifactId>maven-source-plugin</artifactId>
+                    <version>${maven-source-plugin.version}</version>
+                </plugin>
+                <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
+                    <artifactId>maven-javadoc-plugin</artifactId>
+                    <version>${maven-javadoc-plugin.version}</version>
+                </plugin>
+                <plugin>
+                    <groupId>org.codehaus.plexus</groupId>
+                    <artifactId>plexus-component-metadata</artifactId>
+                    <version>${plexus-component-metadata.version}</version>
+                </plugin>
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-enforcer-plugin</artifactId>