Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.

Bump all the things #50

Merged
merged 1 commit into from
Oct 17, 2021
Merged

Bump all the things #50

merged 1 commit into from
Oct 17, 2021

Conversation

cortinico
Copy link
Member

Let's keep the compiler plugins up to date. Specifically I'm bumping to:

assertj = "3.21.0"
detekt = "1.18.1"
kotlin = "1.5.31"
kotlinCompileTesting = "1.4.5"
spek = "2.0.17"

And Gradle to 7.2

@cortinico cortinico requested a review from 3flex October 17, 2021 16:32
@@ -1,5 +1,5 @@
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-7.0.2-all.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-7.2-all.zip
Copy link

Choose a reason for hiding this comment

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

Shouldn't here be distribution type bin instead of all? To make Gradle downloading slightly faster on CI and reduce cache size?

Copy link
Member

Choose a reason for hiding this comment

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

Because this is a plugin project, all is generally more reasonable since we need IntelliJ to be able to navigate the gradle APIs.

@cortinico cortinico merged commit 47ec47b into main Oct 17, 2021
@cortinico cortinico deleted the nc/bump-all-the-things branch October 17, 2021 22:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants