Skip to content

Conversation

fzhinkin
Copy link
Collaborator

@fzhinkin fzhinkin commented Jul 4, 2024

The latest BCV versions are incompatible with LV 2.1 due to the old kotlinx-metadata-jvm version being used.
Unfortunately, BCV could not be easily updated (Kotlin/binary-compatibility-validator#241 (comment)).
This PR changes the BCV for kotlin-commuinity/dev builds to a snapshot build with ab updated metadata version (built from kotlin-community/stable-metadata branch).

@fzhinkin fzhinkin requested review from tbogdanova and qurbonzoda July 4, 2024 13:31
plugins {
id("kotlinx.team.infra") version "0.4.0-dev-80"
id("org.jetbrains.kotlinx.binary-compatibility-validator") version "0.13.2"
id("org.jetbrains.kotlinx.binary-compatibility-validator") version "0.15.0-20240704150100-2.1-SNAPSHOT"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
id("org.jetbrains.kotlinx.binary-compatibility-validator") version "0.15.0-20240704150100-2.1-SNAPSHOT"
id("org.jetbrains.kotlinx.binary-compatibility-validator") version "0.16.0"

Copy link
Contributor

Choose a reason for hiding this comment

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

Or we can merge #180 instead.

Comment on lines +8 to +9
maven { url 'https://maven.pkg.jetbrains.space/kotlin/p/kotlinx/dev'}

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
maven { url 'https://maven.pkg.jetbrains.space/kotlin/p/kotlinx/dev'}

@Goooler
Copy link
Contributor

Goooler commented Jul 24, 2024

This could be closed for now. Handled by #188.

@qurbonzoda qurbonzoda force-pushed the kotlin-community/dev branch from 3b28c01 to 38454d3 Compare August 23, 2024 10:27
@qurbonzoda qurbonzoda closed this Sep 4, 2024
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.

4 participants