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

misc: Upgrade to Kotlin 2.1.0 #1196

Merged
merged 13 commits into from
Dec 17, 2024
Merged

misc: Upgrade to Kotlin 2.1.0 #1196

merged 13 commits into from
Dec 17, 2024

Conversation

lauzadis
Copy link
Contributor

Issue #

Description of changes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@lauzadis lauzadis requested a review from a team as a code owner December 16, 2024 16:01
kotlin-compile-testing = {module = "com.github.tschuchortdev:kotlin-compile-testing", version.ref = "kotlin-compile-testing-version" }
kotlin-compile-testing = {module = "dev.zacsweers.kctfork:core", version.ref = "kotlin-compile-testing-version" }
Copy link
Contributor

Choose a reason for hiding this comment

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

Question: Why are we changing this dependency?

Copy link
Contributor Author

@lauzadis lauzadis Dec 16, 2024

Choose a reason for hiding this comment

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

We're frequently blocked on com.github.tschuchortdev:kotlin-compile-testing because its Kotlin version is not upgraded often enough, leading to mismatched Kotlin compiler errors.

tschuchortdev/kotlin-compile-testing#411

dev.zacsweers.kctfork:core is a fork of the project that's updated more frequently:

NOTE This project is a fork of the original tschuchortdev/kotlin-compile-testing, which itself started as a fork of Moshi's compile test infra. The goal of this fork is to better track the latest Kotlin releases.

We can either take this fork or disable the tests.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh okay, I think taking the fork is fine

This comment has been minimized.

This comment has been minimized.

1 similar comment
Copy link

Affected Artifacts

Changed in size
Artifact Pull Request (bytes) Latest Release (bytes) Delta (bytes) Delta (percentage)
http-client-engine-okhttp-jvm.jar 115,535 115,427 108 0.09%
aws-signing-crt-jvm.jar 15,638 15,624 14 0.09%
logging-slf4j2-jvm.jar 22,141 22,124 17 0.08%
http-client-engine-default-jvm.jar 14,562 14,552 10 0.07%
test-suite-jvm.jar 97,206 97,180 26 0.03%
aws-event-stream-jvm.jar 84,202 84,190 12 0.01%
aws-signing-tests-jvm.jar 456,627 456,568 59 0.01%
http-client-engine-crt-jvm.jar 51,937 51,945 -8 -0.02%
http-jvm.jar 103,431 103,477 -46 -0.04%
http-test-jvm.jar 58,690 58,718 -28 -0.05%
serde-form-url-jvm.jar 35,365 35,382 -17 -0.05%
aws-protocol-core-jvm.jar 20,404 20,414 -10 -0.05%
http-auth-jvm.jar 12,939 12,947 -8 -0.06%
aws-credentials-jvm.jar 24,046 24,061 -15 -0.06%
http-client-jvm.jar 314,610 314,811 -201 -0.06%
telemetry-provider-otel-jvm.jar 30,186 30,209 -23 -0.08%
aws-xml-protocols-jvm.jar 16,982 16,995 -13 -0.08%
runtime-core-jvm.jar 810,542 811,179 -637 -0.08%
telemetry-api-jvm.jar 116,800 116,898 -98 -0.08%
identity-api-jvm.jar 29,469 29,494 -25 -0.08%
aws-json-protocols-jvm.jar 6,897 6,903 -6 -0.09%
testing-jvm.jar 8,220 8,229 -9 -0.11%
serde-xml-jvm.jar 176,853 177,087 -234 -0.13%
serde-cbor-jvm.jar 104,247 104,397 -150 -0.14%
http-auth-aws-jvm.jar 24,637 24,675 -38 -0.15%
serde-json-jvm.jar 70,203 70,338 -135 -0.19%
aws-signing-common-jvm.jar 66,358 66,491 -133 -0.20%
smithy-client-jvm.jar 62,553 62,681 -128 -0.20%
http-auth-api-jvm.jar 6,555 6,569 -14 -0.21%
telemetry-provider-micrometer-jvm.jar 22,785 22,834 -49 -0.21%
serde-jvm.jar 50,977 51,230 -253 -0.49%
aws-signing-default-jvm.jar 51,806 52,433 -627 -1.20%
smithy-test-jvm.jar 62,042 62,937 -895 -1.42%

@lauzadis lauzadis changed the base branch from main to v1.4 December 16, 2024 18:40
@lauzadis lauzadis merged commit 53f23fc into v1.4 Dec 17, 2024
10 checks passed
@lauzadis lauzadis deleted the misc-kotlin-2.1.0 branch December 17, 2024 15:46
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