Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,10 @@
<jacoco.version>0.8.12</jacoco.version>
<jspecify.version>1.0.1</jspecify.version>
<spotless.version>2.43.0</spotless.version>
<!-- Dokka is this project's javadoc, so the standard switch skips it too: a build that asked for
no documentation should not load a documentation engine, let alone a second Kotlin generation
into the same JVM as the compiler. -->
<maven.javadoc.skip>false</maven.javadoc.skip>
</properties>
<groupId>st.orm</groupId>
<artifactId>storm-framework</artifactId>
Expand Down
6 changes: 6 additions & 0 deletions storm-kotlin-spring/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,12 @@
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-maven-plugin</artifactId>
<version>${kotlin.version}</version>
<!-- Contributes the plugin's lifecycle mapping, which synthesises default-compile and
default-kapt executions beside the ones declared below. The duplicate compilation is what
this flag costs, and the flag has to stay: without it, CodeQL's Kotlin extractor binds
against the wrong compiler generation and fails its analysis on a later module in the
reactor, with an AbstractMethodError on ComponentRegistrar. -->
<extensions>true</extensions>
<configuration>
<jvmTarget>21</jvmTarget>
<args>
Expand Down
7 changes: 7 additions & 0 deletions storm-ktor-test/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,12 @@
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-maven-plugin</artifactId>
<version>${kotlin.version}</version>
<!-- Contributes the plugin's lifecycle mapping, which synthesises default-compile and
default-kapt executions beside the ones declared below. The duplicate compilation is what
this flag costs, and the flag has to stay: without it, CodeQL's Kotlin extractor binds
against the wrong compiler generation and fails its analysis on a later module in the
reactor, with an AbstractMethodError on ComponentRegistrar. -->
<extensions>true</extensions>
<configuration>
<jvmTarget>21</jvmTarget>
<args>
Expand Down Expand Up @@ -149,6 +155,7 @@
roots, which include Maven's src/main/java whether it exists or not. Naming the Kotlin root
keeps a missing directory from being reported as an error on every build. -->
<configuration>
<skip>${maven.javadoc.skip}</skip>
<sourceDirectories>
<dir>${project.basedir}/src/main/kotlin</dir>
</sourceDirectories>
Expand Down
7 changes: 7 additions & 0 deletions storm-ktor/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,12 @@
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-maven-plugin</artifactId>
<version>${kotlin.version}</version>
<!-- Contributes the plugin's lifecycle mapping, which synthesises default-compile and
default-kapt executions beside the ones declared below. The duplicate compilation is what
this flag costs, and the flag has to stay: without it, CodeQL's Kotlin extractor binds
against the wrong compiler generation and fails its analysis on a later module in the
reactor, with an AbstractMethodError on ComponentRegistrar. -->
<extensions>true</extensions>
<configuration>
<jvmTarget>21</jvmTarget>
<args>
Expand Down Expand Up @@ -177,6 +183,7 @@
roots, which include Maven's src/main/java whether it exists or not. Naming the Kotlin root
keeps a missing directory from being reported as an error on every build. -->
<configuration>
<skip>${maven.javadoc.skip}</skip>
<sourceDirectories>
<dir>${project.basedir}/src/main/kotlin</dir>
</sourceDirectories>
Expand Down
Loading