Skip to content

fix: kotlin compatibility with gradle plugin#919

Merged
dieppa merged 2 commits into
release/1.4from
issue/#918
Jun 2, 2026
Merged

fix: kotlin compatibility with gradle plugin#919
dieppa merged 2 commits into
release/1.4from
issue/#918

Conversation

@osantana85

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the Flamingock Gradle plugin’s Kotlin support by ensuring annotation processing runs correctly for Kotlin JVM projects, and by hardening reflective compiler-argument configuration. It also bumps the root snapshot version.

Changes:

  • Auto-applies the Kotlin KAPT plugin when org.jetbrains.kotlin.jvm is present so Flamingock’s annotation processor runs for Kotlin sources.
  • Adds Flamingock processor (and Mongock support processor when enabled) to the kapt dependency configuration when KAPT is available.
  • Fixes reflective invocation in compiler-args configuration to better handle array/vararg signatures for Kotlin compatibility.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
flamingock-gradle-plugin/src/main/kotlin/io/flamingock/gradle/internal/DependencyConfigurator.kt Adds KAPT-aware dependency wiring for Flamingock (and Mongock support) processors.
flamingock-gradle-plugin/src/main/kotlin/io/flamingock/gradle/internal/CompilerArgsConfigurator.kt Adjusts reflective method invocation to better match Kotlin/Gradle signature shapes.
flamingock-gradle-plugin/src/main/kotlin/io/flamingock/gradle/FlamingockPlugin.kt Auto-applies KAPT for Kotlin JVM projects to preserve “zero boilerplate” behavior.
build.gradle.kts Bumps declared snapshot version to 1.4.1-SNAPSHOT.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

@dieppa dieppa merged commit 735f98d into release/1.4 Jun 2, 2026
5 checks passed
@dieppa dieppa deleted the issue/#918 branch June 2, 2026 08:49
dieppa added a commit that referenced this pull request Jun 2, 2026
Co-authored-by: Oliver Santana <oliver.sm@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants