Skip to content

Conversation

@alexander-yevsyukov
Copy link
Contributor

This PR adds the Project.addUserClasspathDependency() extension which was previously adopted in CoreJvm Compiler. The extension now is also needed for the Validation Compiler Gradle Plugin.

Copilot AI review requested due to automatic review settings December 8, 2025 21:00
@alexander-yevsyukov alexander-yevsyukov self-assigned this Dec 8, 2025
Copy link
Contributor

Copilot AI left a comment

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 adds the Project.addUserClasspathDependency() extension function to the Gradle API module, making it available for reuse across multiple compiler plugins. The function was previously used in CoreJvm Compiler and is now needed for the Validation Compiler Gradle Plugin.

Key changes:

  • Added public extension function addUserClasspathDependency() that accepts varargs of MavenArtifact objects
  • Added private helper functions addDependency() and findDependency() to support the main function
  • Bumped version from 2.0.0-SNAPSHOT.034 to 2.0.0-SNAPSHOT.035

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
version.gradle.kts Version bump to 2.0.0-SNAPSHOT.035
pom.xml Version bump to match gradle.kts
gradle-api/src/main/kotlin/io/spine/tools/compiler/gradle/api/ProjectExts.kt Added addUserClasspathDependency() extension and supporting functions with necessary imports
dependencies.md Auto-generated dependency report with updated version references and timestamps

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

@alexander-yevsyukov alexander-yevsyukov merged commit 3005c2d into master Dec 8, 2025
12 checks passed
@alexander-yevsyukov alexander-yevsyukov deleted the add-user-classpath-ext branch December 8, 2025 21:20
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