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

Enable KLib ABI validation #402

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

@samhill303 samhill303 requested a review from kpgalligan August 20, 2024 14:17
@Goooler Goooler force-pushed the enable-klib-validation branch from 6c3bdca to f51a77f Compare August 20, 2024 14:36
}

rootProject.name = "kermit-root"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed? What is this added for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, there is a bug in BCV related to Kotlin/binary-compatibility-validator#261.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ooops, it still failed on CI, see https://github.com/touchlab/Kermit/actions/runs/11709073063/job/32612290197.

May have to revert this change until the issue has been fixed on BCV side.

@KevinSchildhorn KevinSchildhorn merged commit 4a0ea16 into touchlab:main Nov 6, 2024
@Goooler Goooler deleted the enable-klib-validation branch November 7, 2024 01:46
KevinSchildhorn added a commit that referenced this pull request Nov 7, 2024
KevinSchildhorn added a commit that referenced this pull request Nov 7, 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.

2 participants