Skip to content

Apply kotlin version compat#9322

Merged
yschimke merged 3 commits into
lysine-dev:masterfrom
yschimke:kotlin_compat
Feb 9, 2026
Merged

Apply kotlin version compat#9322
yschimke merged 3 commits into
lysine-dev:masterfrom
yschimke:kotlin_compat

Conversation

@yschimke

@yschimke yschimke commented Feb 8, 2026

Copy link
Copy Markdown
Collaborator

Copy the approach from Okio.

lysine-dev/okio#1770

Closes #9032

Copy the approach from Okio.
@yschimke

yschimke commented Feb 8, 2026

Copy link
Copy Markdown
Collaborator Author

@JakeWharton is this correct for matching Okio?

@JakeWharton JakeWharton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me!

Comment thread build-logic/src/main/kotlin/okhttp.jvm-conventions.gradle.kts Outdated
Comment thread gradle/libs.versions.toml

# Set to lower version than KGP version to maximize compatibility
# Default to matching okio
kotlinCoreLibrariesVersion = "2.1.21"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there a reason to define this here rather than at the sole usage site?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Consistency of versions and copying okio setup?

Co-authored-by: Jake Wharton <jw@jakewharton.com>
@yschimke
yschimke merged commit e08bac1 into lysine-dev:master Feb 9, 2026
20 of 22 checks passed

@swankjesse swankjesse left a comment

Copy link
Copy Markdown
Collaborator

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

Development

Successfully merging this pull request may close these issues.

Incompatible Kotlin Version Error with OkHttp v5.1.0

3 participants