Skip to content

Conversation

Bencodes
Copy link
Collaborator

@Bencodes Bencodes commented Apr 9, 2025

is KotlinJvmBinarySourceElement -> binaryClass.location
is JvmPackagePartSource -> this.knownJvmBinaryClass?.location
is JavaBinarySourceElement -> this.javaClass.virtualFile.path
is VirtualFileBasedSourceElement -> this.virtualFile.path
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is a breaking change, we might be able to work around this with some less-than-amazing reflection in the short term but it'll be pretty ugly.

@Bencodes Bencodes force-pushed the add-support-for-kotlin-2.1.20 branch from 93bf53e to 8883ad2 Compare April 18, 2025 21:37
@restingbull
Copy link
Collaborator

They really aren't making it easy on us.

@Bencodes Bencodes force-pushed the add-support-for-kotlin-2.1.20 branch from 4c8adb8 to e02a8be Compare May 28, 2025 21:05
@Bencodes Bencodes force-pushed the add-support-for-kotlin-2.1.20 branch from 21ae65f to 32eb56f Compare May 29, 2025 22:11
@Bencodes Bencodes merged commit 2d41fbc into master May 29, 2025
4 checks passed
oliviernotteghem pushed a commit to uber-common/rules_kotlin that referenced this pull request May 30, 2025
* Add support for Kotlin 2.1.20

* Docs

* Fixes

* Fix maven_install file name for 2.1.20 version

* Get the compose example building again
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