JetBrains / intellij-platform-plugin-template

Template repository for creating plugins for IntelliJ Platform
Apache License 2.0
3.05k stars 618 forks source link

(🎁) Migrate to Gradle Kotlin property assignment #357

Closed KotlinIsland closed 1 year ago

KotlinIsland commented 1 year ago

Hmm, where did the next branch go?πŸ•΅οΈβ€β™‚οΈ

This is draft until Gradle 8.1 is released.

codecov-commenter commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (next@139730d). Click here to learn what that means. Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## next #357 +/- ## ======================================= Coverage ? 26.66% ======================================= Files ? 4 Lines ? 30 Branches ? 0 ======================================= Hits ? 8 Misses ? 22 Partials ? 0 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JetBrains). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JetBrains)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

hsz commented 1 year ago

next is already here.

Wow, this assignment is hot β€” finally! Thanks for the pr. 😊

KotlinIsland commented 1 year ago

@hsz We could merge this now and just not merge next into main until 8.1, or we could keep this as draft and only merge it into next when 8.1 is released. What do you think?

hsz commented 1 year ago

Let's keep it as it is util Gradle 8.1 becomes GA. You never know if a hotfix will appear in the meantime or not.

hsz commented 1 year ago

8.1 is here, merged. Thanks!