Skip to content

Enable KLib ABI validation#402

Merged
KevinSchildhorn merged 1 commit intotouchlab:mainfrom
Goooler:enable-klib-validation
Nov 6, 2024
Merged

Enable KLib ABI validation#402
KevinSchildhorn merged 1 commit intotouchlab:mainfrom
Goooler:enable-klib-validation

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
} No newline at end of file
}

rootProject.name = "kermit-root" No newline at end of file
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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