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

chore(deps): Bump kork and Kotlin #1841

Merged
merged 2 commits into from
Mar 3, 2021
Merged

Conversation

luispollo
Copy link
Contributor

Bumps kork to pick up spinnaker/kork#850 and aligns Kotlin version in keel.

@luispollo luispollo requested a review from robfletcher March 3, 2021 19:23
@@ -17,8 +17,8 @@ buildscript {
}

plugins {
kotlin("jvm") version "1.4.21" apply false
id("org.jetbrains.kotlin.plugin.allopen") version "1.4.21" apply false
kotlin("jvm") version "1.4.31" apply false
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@robfletcher Is there a way we could avoid specifying the version here and just pick up the version defined in kork?

Copy link
Contributor

Choose a reason for hiding this comment

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

No

Copy link
Contributor Author

Choose a reason for hiding this comment

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

K

Copy link
Contributor

Choose a reason for hiding this comment

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

It's annoying but it's not possible to specify Gradle plugin versions with a variable / property because they get loaded before anything else (including project properties). It's a long-standing gripe.

@mergify mergify bot merged commit 125c162 into spinnaker:master Mar 3, 2021
@mergify mergify bot added the auto merged label Mar 3, 2021
@luispollo luispollo deleted the chores branch March 3, 2021 21:38
osoriano pushed a commit to osoriano/keel that referenced this pull request Sep 2, 2023
* chore(deps): Bump kork and Kotlin

* fix(pr): Fix compile errors with new kotlin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged ready to merge Approved and ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants