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

Add the kotlinx binary compatibility plugin #813

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

RBusarow
Copy link
Collaborator

Copy link
Member

@JoelWilcox JoelWilcox left a comment

Choose a reason for hiding this comment

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

Very cool! First time seeing this plugin

Copy link
Member

Choose a reason for hiding this comment

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

Debating whether we will want to disable it for this module since we don't guarantee API compatibility on this and more-so provide it for convenience, but makes sense to start enabled and see whether we end up with noise problems.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Those were pretty much my thoughts as well. My rationale for leaving it enabled is that even though we don't guarantee anything, it's still nice to know? Unless it gets annoying.

Copy link
Member

Choose a reason for hiding this comment

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

Yup I agree completely! And it's easy remove later if it does get annoying

@RBusarow RBusarow force-pushed the rick/binary-compatibility branch from 8dcf24a to 2cf82e4 Compare December 15, 2023 00:16
@RBusarow RBusarow enabled auto-merge December 15, 2023 00:17
@RBusarow RBusarow merged commit a72d46f into main Dec 15, 2023
23 checks passed
@RBusarow RBusarow deleted the rick/binary-compatibility branch December 15, 2023 00:29
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