Skip to content

Commit 4653947

Browse files
committed
Fix #274: add proper module-info.java handling
1 parent f183a99 commit 4653947

File tree

4 files changed

+6
-13
lines changed

4 files changed

+6
-13
lines changed

pom.xml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ https://stackoverflow.com/questions/16302247/maven-property-project-artifact-sel
6262
<dependency>
6363
<groupId>org.junit.jupiter</groupId>
6464
<artifactId>junit-jupiter</artifactId>
65-
<version>${version.junit5}</version>
6665
<scope>test</scope>
6766
</dependency>
6867
</dependencies>
@@ -99,11 +98,6 @@ https://stackoverflow.com/questions/16302247/maven-property-project-artifact-sel
9998

10099
<build>
101100
<plugins>
102-
<plugin>
103-
<groupId>org.moditect</groupId>
104-
<artifactId>moditect-maven-plugin</artifactId>
105-
</plugin>
106-
107101
<!-- 08-Nov-2019, tatu: Copied from
108102
https://github.com/stephenc/git-timestamp-maven-plugin/blob/master/pom.xml#L327-L337
109103
-->

release-notes/VERSION

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,5 @@ mostly)
1212
3.0 (not yet released)
1313

1414
#272: Require JDK17 for jackson-annotations 3.0 for proper `module-info.java`
15+
#274: Change 3.0 to use `module-info.java` directly for build (instead of via Moditect)
1516
- Change `JsonInclude.value`, `JsonInclude.content` to `USE_DEFAULTS` (from `ALWAYS`)

src/main/java/module-info.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
// Jackson 3.x module-info for jackson-annotations Main artifact
2+
module com.fasterxml.jackson.annotation {
3+
// Might not be required beyond exports but just in case:
4+
opens com.fasterxml.jackson.annotation;
5+
}

src/moditect/module-info.java

Lines changed: 0 additions & 7 deletions
This file was deleted.

0 commit comments

Comments
 (0)