Browse Source

Add module-info support and run CI with Java 11 (#1948)

* Adds explicit module descriptors to a subset of modules (#1943)

Fixes #1357

* Configure CI to run with Java 11

* Configure moditect for all modules, enable only on those that required it

* Do not skip moditect when running tests

* Only skip modules that don't work with moditect plugin

---------

Co-authored-by: Marvin Froeder <velo@users.noreply.github.com>
Co-authored-by: Marvin Froeder <velobr@gmail.com>
pull/1876/head
Andres Almiray 2 years ago committed by GitHub
parent
commit
191fea1464
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      apt-test-generator/pom.xml
  2. 2
      benchmark/pom.xml
  3. 2
      jackson-jaxb/pom.xml
  4. 7
      jaxrs2/pom.xml
  5. 37
      pom.xml
  6. 2
      ribbon/pom.xml
  7. 2
      soap/pom.xml

2
apt-test-generator/pom.xml

@ -30,6 +30,8 @@ @@ -30,6 +30,8 @@
<properties>
<main.basedir>${project.basedir}/..</main.basedir>
<moditect.skip>true</moditect.skip>
</properties>
<dependencies>

2
benchmark/pom.xml

@ -32,6 +32,8 @@ @@ -32,6 +32,8 @@
<rx.java.version>1.3.8</rx.java.version>
<netty.version>4.1.89.Final</netty.version>
<main.basedir>${project.basedir}/..</main.basedir>
<moditect.skip>true</moditect.skip>
</properties>
<dependencyManagement>

2
jackson-jaxb/pom.xml

@ -29,6 +29,8 @@ @@ -29,6 +29,8 @@
<properties>
<main.basedir>${project.basedir}/..</main.basedir>
<moditect.skip>true</moditect.skip>
</properties>
<dependencies>

7
jaxrs2/pom.xml

@ -28,12 +28,9 @@ @@ -28,12 +28,9 @@
<description>Feign JAX-RS 2</description>
<properties>
<!-- override default bytecode version for src/main from parent pom -->
<main.java.version>1.8</main.java.version>
<main.signature.artifact>java18</main.signature.artifact>
<main.basedir>${project.basedir}/..</main.basedir>
<maven.compiler.source>1.8</maven.compiler.source>
<maven.compiler.target>1.8</maven.compiler.target>
<moditect.skip>true</moditect.skip>
</properties>
<dependencies>

37
pom.xml

@ -66,6 +66,7 @@ @@ -66,6 +66,7 @@
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<project.build.resourceEncoding>UTF-8</project.build.resourceEncoding>
<moditect.skip>false</moditect.skip>
<!-- specifying jvm arguments -->
<jvm.options>-Duser.language=en</jvm.options>
@ -112,6 +113,7 @@ @@ -112,6 +113,7 @@
<maven-gpg-plugin.version>3.0.1</maven-gpg-plugin.version>
<maven-deploy-plugin.version>3.1.0</maven-deploy-plugin.version>
<docker-maven-plugin.version>1.2.2</docker-maven-plugin.version>
<moditect-maven-plugin.version>1.0.0.RC2</moditect-maven-plugin.version>
</properties>
<url>https://github.com/openfeign/feign</url>
<inceptionYear>2012</inceptionYear>
@ -460,6 +462,36 @@ @@ -460,6 +462,36 @@
</configuration>
</plugin>
<plugin>
<groupId>org.moditect</groupId>
<artifactId>moditect-maven-plugin</artifactId>
<version>${moditect-maven-plugin.version}</version>
<executions>
<execution>
<id>add-module-infos</id>
<phase>package</phase>
<goals>
<goal>add-module-info</goal>
</goals>
<configuration>
<skip>${moditect.skip}</skip>
<overwriteExistingFiles>true</overwriteExistingFiles>
<module>
<moduleInfo>
<!-- module name will be derived from filename -->
<!-- export everything -->
<exports>*;</exports>
<!-- declare services consumed by the artifact -->
<addServiceUses>true</addServiceUses>
</moduleInfo>
</module>
<jdepsExtraArgs>
<arg>--multi-release=9</arg>
</jdepsExtraArgs>
</configuration>
</execution>
</executions>
</plugin>
</plugins>
</pluginManagement>
@ -498,6 +530,11 @@ @@ -498,6 +530,11 @@
</executions>
</plugin>
<plugin>
<groupId>org.moditect</groupId>
<artifactId>moditect-maven-plugin</artifactId>
</plugin>
<plugin>
<artifactId>maven-compiler-plugin</artifactId>
<inherited>true</inherited>

2
ribbon/pom.xml

@ -30,6 +30,8 @@ @@ -30,6 +30,8 @@
<properties>
<main.basedir>${project.basedir}/..</main.basedir>
<ribbon-version>2.7.18</ribbon-version>
<moditect.skip>true</moditect.skip>
</properties>
<dependencyManagement>

2
soap/pom.xml

@ -29,6 +29,8 @@ @@ -29,6 +29,8 @@
<properties>
<main.basedir>${project.basedir}/..</main.basedir>
<moditect.skip>true</moditect.skip>
</properties>
<dependencies>

Loading…
Cancel
Save