Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Kotlin 1.9.x #75

Merged
merged 1 commit into from
Feb 2, 2024
Merged

Conversation

olafurpg
Copy link
Member

@olafurpg olafurpg commented Feb 2, 2024

Fixes #74. My limited understanding of Kotlin compiler compatibility is that this change drops support for 1.8.x. Until we move to the more stable Kotlin compiler API (which I have barely researched), I think it's reasonable to target the latest Kotlin version in scip-kotlin. Users on older versions of Kotlin will need to stay on older versions of scip-kotlin.

Test plan

Green CI.

Fixes sourcegraph#74. My limited understanding of Kotlin compiler compatibility is
that this change drops support for 1.8.x. Until we move to the more
stable Kotlin compiler API (which I have barely researched), I think
it's reasonable to target the latest Kotlin version in scip-kotlin.
Users on older versions of Kotlin will need to stay on older versions of
scip-kotlin.
| Kotlin version | scip-kotlin version |
|----------------|---------------------|
| 1.8.x | 0.3.2 |
| 1.9.x | 0.4.0 |
Copy link
Member Author

Choose a reason for hiding this comment

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

0.4.0 is not released yet but I can cut it after we merge this PR

@olafurpg olafurpg merged commit 3ff9e97 into sourcegraph:main Feb 2, 2024
4 checks passed
@olafurpg olafurpg deleted the olafurpg/kotlin-1.9 branch February 2, 2024 10:50
Copy link

@pkukielka pkukielka left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants