Skip to content

Commit

Permalink
Use latest Maven version
Browse files Browse the repository at this point in the history
Similar to PRs asciidoctor#717 and asciidoctor#719, to ease project maintenance
we can automate Maven (core and transitives) dependencies
and use CI to validate backward compatibility.
* Remove maven exclusion from Dependabot.
* Deleted direct references to dependencies that are provided
by maven-core: ideally we'd like to refer them if we use them,
but ther lack of a BOM make it un-practical.
* Rename build properties for ease of use.
* Fix mising 'url' in 'asciidoctor-parser-doxia-module' module.
  • Loading branch information
abelsromero committed Jan 1, 2024
1 parent 7a02121 commit 69a0a5d
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 51 deletions.
1 change: 0 additions & 1 deletion .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ updates:
target-branch: "main"
ignore:
- dependency-name: org.jruby:jruby
- dependency-name: org.apache.maven:*
- dependency-name: org.glassfish.jaxb:jaxb-runtime
update-types:
- "version-update:semver-major"
Expand Down
2 changes: 1 addition & 1 deletion asciidoctor-converter-doxia-module/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
<dependency>
<groupId>org.apache.maven</groupId>
<artifactId>maven-core</artifactId>
<version>${maven.project.version}</version>
<version>${maven.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
Expand Down
8 changes: 2 additions & 6 deletions asciidoctor-maven-commons/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,6 @@
</description>
<url>https://github.com/asciidoctor/asciidoctor-maven-plugin</url>

<properties>
<plexus.utils.version>3.5.1</plexus.utils.version>
</properties>

<dependencies>
<dependency>
<groupId>org.asciidoctor</groupId>
Expand All @@ -35,13 +31,13 @@
<dependency>
<groupId>org.apache.maven</groupId>
<artifactId>maven-core</artifactId>
<version>${maven.project.version}</version>
<version>${maven.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-utils</artifactId>
<version>${plexus.utils.version}</version>
<version>3.5.1</version>
<scope>compile</scope>
</dependency>
</dependencies>
Expand Down
44 changes: 3 additions & 41 deletions asciidoctor-maven-plugin/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@

<properties>
<maven-plugin-tools.version>3.10.2</maven-plugin-tools.version>
<netty.version>4.1.104.Final</netty.version>
</properties>

<dependencies>
Expand All @@ -44,8 +43,8 @@
</dependency>
<dependency>
<groupId>org.apache.maven</groupId>
<artifactId>maven-plugin-api</artifactId>
<version>${maven.project.version}</version>
<artifactId>maven-core</artifactId>
<version>${maven.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
Expand All @@ -54,17 +53,6 @@
<version>${maven-plugin-tools.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.apache.maven</groupId>
<artifactId>maven-core</artifactId>
<version>${maven.project.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-interpolation</artifactId>
<version>1.26</version>
</dependency>
<dependency>
<groupId>commons-io</groupId>
<artifactId>commons-io</artifactId>
Expand All @@ -73,7 +61,7 @@
<dependency>
<groupId>io.netty</groupId>
<artifactId>netty-codec-http</artifactId>
<version>${netty.version}</version>
<version>4.1.104.Final</version>
</dependency>
</dependencies>

Expand All @@ -96,32 +84,6 @@
<skipErrorNoDescriptorsFound>true</skipErrorNoDescriptorsFound>
</configuration>
</plugin>
<!--This plugin's configuration is used to store Eclipse m2e settings only. It has no influence on the Maven build itself.-->
<plugin>
<groupId>org.eclipse.m2e</groupId>
<artifactId>lifecycle-mapping</artifactId>
<version>1.0.0</version>
<configuration>
<lifecycleMappingMetadata>
<pluginExecutions>
<pluginExecution>
<pluginExecutionFilter>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-plugin-plugin</artifactId>
<versionRange>[${maven.plugin.plugin.version},)</versionRange>
<goals>
<goal>descriptor</goal>
<goal>helpmojo</goal>
</goals>
</pluginExecutionFilter>
<action>
<execute/>
</action>
</pluginExecution>
</pluginExecutions>
</lifecycleMappingMetadata>
</configuration>
</plugin>
</plugins>
</pluginManagement>
</build>
Expand Down
3 changes: 2 additions & 1 deletion asciidoctor-parser-doxia-module/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@

<name>Asciidoctor Parser Doxia Module</name>
<description>Asciidoctor Doxia Module based on Asciidoctor AST processing (for Maven Site integration)</description>
<url>https://github.com/asciidoctor/asciidoctor-maven-plugin</url>

<dependencies>
<dependency>
Expand All @@ -35,7 +36,7 @@
<dependency>
<groupId>org.apache.maven</groupId>
<artifactId>maven-core</artifactId>
<version>${maven.project.version}</version>
<version>${maven.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
<properties>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<project.java.version>11</project.java.version>
<maven.project.version>3.9.1</maven.project.version>
<maven.version>3.9.6</maven.version>
<doxia.version>1.12.0</doxia.version>
<plexus-component-metadata.version>2.2.0</plexus-component-metadata.version>
<asciidoctorj.version>2.5.11</asciidoctorj.version>
Expand Down

0 comments on commit 69a0a5d

Please sign in to comment.