Skip to content

Conversation

Goooler
Copy link
Member

@Goooler Goooler commented Apr 23, 2025

@Goooler
Copy link
Member Author

Goooler commented Apr 23, 2025

@Goooler Goooler force-pushed the g/20250423/test-older-kmp branch from 09e15dc to 3e56d26 Compare April 23, 2025 04:49
This reverts commit 3e56d26.
@Goooler Goooler changed the title Test for KMP 2.0.21-RC Set Main-Class attr for KMP 1.9.0 or above Apr 23, 2025
@Goooler Goooler marked this pull request as ready for review April 23, 2025 05:16
@Goooler Goooler requested a review from Copilot April 23, 2025 05:16
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 updates the minimum supported KMP version for setting the Main-Class attribute to 1.9.0. The changes include updating the version check in the plugin code, removing the outdated version reference in the documentation, and adding a corresponding changelog entry.

Reviewed Changes

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

File Description
src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/ShadowKmpPlugin.kt Updated version check condition to align with KMP 1.9.0+ requirements.
docs/kotlin-plugins/README.md Removed outdated version reference and updated code snippet comments.
docs/changes/README.md Added changelog entry for setting the Main-Class attribute for KMP 1.9.0 or above.
Comments suppressed due to low confidence (1)

docs/kotlin-plugins/README.md:125

  • Kotlin string literals require double quotes. Please update 'myapp.MainKt' to "myapp.MainKt" to ensure correct syntax in the code snippet.
it.mainClass.set('myapp.MainKt')

@Goooler Goooler merged commit afbae30 into main Apr 23, 2025
9 checks passed
@Goooler Goooler deleted the g/20250423/test-older-kmp branch April 23, 2025 06:11
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.

1 participant