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 kotlin-conventions plugin #619

Merged
merged 1 commit into from
Aug 11, 2021
Merged

Add kotlin-conventions plugin #619

merged 1 commit into from
Aug 11, 2021

Conversation

turansky
Copy link
Collaborator

import org.jetbrains.kotlin.gradle.dsl.KotlinJsProjectExtension
import org.jetbrains.kotlin.gradle.plugin.KotlinJsPluginWrapper

plugins.withType<KotlinJsPluginWrapper> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is it different from the removed one?

Copy link
Collaborator Author

@turansky turansky Aug 11, 2021

Choose a reason for hiding this comment

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

I removed outdated and unused code
Build remains the same

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Convensions plugin - Gradle best practice
We have already had one for publish

@turansky turansky requested a review from Leonya August 11, 2021 19:38
@Leonya Leonya merged commit b02c0bd into JetBrains:master Aug 11, 2021
@turansky turansky deleted the conventions-plugin branch August 11, 2021 23:55
@Leonya
Copy link
Collaborator

Leonya commented Aug 12, 2021

Released in pre.230

@turansky turansky mentioned this pull request Aug 27, 2021
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