Skip to content

Conversation

alexander-yevsyukov
Copy link
Contributor

@alexander-yevsyukov alexander-yevsyukov commented Oct 13, 2025

This PR migrates the build and runtime dependencies of CoreJvm Compiler to Gradle 9.1.0. Kotlin was bumped to version 2.2.20. Other dependencies were bumped with applying latest config.

Task dependencies that are now required by Gradle are added manually. Their automatic configuration by CoreJvm Compiler would be added by one of the following PRs.

@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 17:26
@alexander-yevsyukov alexander-yevsyukov self-assigned this Oct 13, 2025
Copy link
Contributor

@Copilot 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 sets missing task dependencies and updates various library versions. The main focus is ensuring that KSP (Kotlin Symbol Processing) tasks depend on Spine compiler tasks to prevent build order issues.

  • Adds explicit task dependencies between KSP and Spine compiler tasks across multiple modules
  • Updates library versions including Kotlin (2.1.21 → 2.2.20), Gradle (8.14.2 → 9.1.0), and various Spine framework dependencies
  • Refactors Shadow JAR configuration to use append instead of deprecated mergeServiceFiles

Reviewed Changes

Copilot reviewed 41 out of 46 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
Multiple build.gradle.kts files Added afterEvaluate blocks to set KSP task dependencies on Spine compiler tasks
version.gradle.kts Updated core JVM compiler version from 015 to 016
pom.xml Updated multiple dependency versions and reorganized some dependencies
gradle-wrapper.properties Upgraded Gradle from 8.14.2 to 9.1.0
buildSrc files Updated dependency versions and refactored dependency management
plugin-bundle/build.gradle.kts Changed Shadow JAR configuration from mergeServiceFiles to append
Files not reviewed (1)
  • .idea/inspectionProfiles/Project_Default.xml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 17:28
Copy link
Contributor

@Copilot 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

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

Files not reviewed (1)
  • .idea/inspectionProfiles/Project_Default.xml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@alexander-yevsyukov alexander-yevsyukov changed the title Set missing task dependencies Bump Gradle, Kotlin, and other dependencies Oct 13, 2025
@alexander-yevsyukov alexander-yevsyukov marked this pull request as ready for review October 13, 2025 17:31
@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 17:35
Copy link
Contributor

@Copilot 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

Copilot reviewed 40 out of 45 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • .idea/inspectionProfiles/Project_Default.xml: Language not supported
Comments suppressed due to low confidence (1)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@alexander-yevsyukov alexander-yevsyukov merged commit 231997e into master Oct 13, 2025
6 of 7 checks passed
@alexander-yevsyukov alexander-yevsyukov deleted the set-missing-task-dependencies branch October 13, 2025 18:39
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