Skip to content

Refactor: Migration to Kotlin DSL and newer Gradle APIs#22

Merged
kdroidFilter merged 1 commit into
NucleusFramework:mainfrom
tuuhin:ref/kotlin-dsl
May 13, 2026
Merged

Refactor: Migration to Kotlin DSL and newer Gradle APIs#22
kdroidFilter merged 1 commit into
NucleusFramework:mainfrom
tuuhin:ref/kotlin-dsl

Conversation

@tuuhin
Copy link
Copy Markdown
Contributor

@tuuhin tuuhin commented May 13, 2026

🚀 Description

This PR includes migration from java-gradle-plugin to kotlin-dsl for the plugin

Changes

  • Update plugin-build/plugin to use the 'kotlin-dsl' plugin
  • Refactor task registration, extension creation, and type access across the codebase to use reified type parameters (e.g., register, create, getByType).
  • Rename internal properties in GenerateNativeBridgesTask with a 'task' prefix for improved clarity and consistency due to property name conflicts with PSiParseWorkerAction

📄 Motivation and Context

We were using class references with java-gradle-plugin with older gradle apis upgraded to kotlin-dsl with reified type parameters

🧪 How Has This Been Tested?

  • Build Verification: Ran ./gradlew build on projects using standard naming conventions. and run the examples

📦 Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

- Update plugin-build/plugin to use the 'kotlin-dsl' plugin
- Refactor task registration, extension creation, and type access across the codebase to use reified type parameters (e.g., register<T>, create<T>, getByType<T>).
- Migrate internal plugin configurations to the newer Gradle 8.4+ APIs (dependencyScope and resolvable) for better dependency management.
- Rename internal properties in GenerateNativeBridgesTask with a 'task' prefix for improved clarity and consistency due to class conflicts with `PSiParseWorkerAction`
@kdroidFilter kdroidFilter merged commit 5b2ad59 into NucleusFramework:main May 13, 2026
13 of 14 checks passed
@kdroidFilter
Copy link
Copy Markdown
Collaborator

Thanks ! Love it ! :)

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.

2 participants