Skip to content
Closed
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
4 changes: 4 additions & 0 deletions storm-kotlin-spring/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,10 @@
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-maven-plugin</artifactId>
<version>${kotlin.version}</version>
<!-- Contributes the plugin's lifecycle mapping, which also synthesises default-compile and
default-kapt executions beside the ones declared below. That duplication is deliberate for
now: dropping it changed what CodeQL's Kotlin extractor sees and broke its analysis. -->
<extensions>true</extensions>
<configuration>
<jvmTarget>21</jvmTarget>
<args>
Expand Down
8 changes: 5 additions & 3 deletions storm-ktor-test/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,10 @@
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-maven-plugin</artifactId>
<version>${kotlin.version}</version>
<!-- Contributes the plugin's lifecycle mapping, which also synthesises default-compile and
default-kapt executions beside the ones declared below. That duplication is deliberate for
now: dropping it changed what CodeQL's Kotlin extractor sees and broke its analysis. -->
<extensions>true</extensions>
<configuration>
<jvmTarget>21</jvmTarget>
<args>
Expand Down Expand Up @@ -141,14 +145,12 @@
<plugin>
<groupId>org.jetbrains.dokka</groupId>
<artifactId>dokka-maven-plugin</artifactId>
<!-- Dokka 2.0 analyses with a Kotlin 2.0 frontend, which cannot read the Kotlin 2.3 metadata this
module and its stdlib carry; 2.2 can. The rest of the framework stays on 2.0 with its own
compiler generation. -->
<version>2.2.0</version>
<!-- The module has no Java sources, and Dokka's default source roots are the project's compile
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
8 changes: 5 additions & 3 deletions storm-ktor/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-maven-plugin</artifactId>
<version>${kotlin.version}</version>
<!-- Contributes the plugin's lifecycle mapping, which also synthesises default-compile and
default-kapt executions beside the ones declared below. That duplication is deliberate for
now: dropping it changed what CodeQL's Kotlin extractor sees and broke its analysis. -->
<extensions>true</extensions>
<configuration>
<jvmTarget>21</jvmTarget>
<args>
Expand Down Expand Up @@ -169,14 +173,12 @@
<plugin>
<groupId>org.jetbrains.dokka</groupId>
<artifactId>dokka-maven-plugin</artifactId>
<!-- Dokka 2.0 analyses with a Kotlin 2.0 frontend, which cannot read the Kotlin 2.3 metadata this
module and its stdlib carry; 2.2 can. The rest of the framework stays on 2.0 with its own
compiler generation. -->
<version>2.2.0</version>
<!-- The module has no Java sources, and Dokka's default source roots are the project's compile
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