-
Notifications
You must be signed in to change notification settings - Fork 28
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
Migrate build logic to Kotlin DSL #298
Conversation
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
id("org.gradle.toolchains.foojay-resolver-convention") version "0.8.0" | ||
} | ||
|
||
val isCI = System.getenv("GITHUB_ACTIONS") != null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
val isCI = System.getenv("GITHUB_ACTIONS") != null | |
val isCI = providers.environmentVariable("CI").isPresent |
remote(develocity.buildCache) { | ||
isEnabled = true | ||
// Check access key presence to avoid build cache errors on PR builds when access key is not present | ||
val accessKey = System.getenv("GRADLE_ENTERPRISE_ACCESS_KEY") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
val accessKey = System.getenv("GRADLE_ENTERPRISE_ACCESS_KEY") | |
val accessKey = providers.environmentVariable("GRADLE_ENTERPRISE_ACCESS_KEY").isPresent |
} | ||
|
||
githubRelease { | ||
token(System.getenv("CCUD_GIT_TOKEN") ?: "") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
token(System.getenv("CCUD_GIT_TOKEN") ?: "") | |
token(providers.environmentVariable("CCUD_GIT_TOKEN").orNull.orEmpty()) |
Thanks for reviewing @Goooler, I've kept the scope to the minimum required to migrate to Kotlin DSL, but those would be good contributions for another PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @3flex! I made one suggestion. If you take that and fix the merge conflict, we'd be happy to accept this.
Co-authored-by: Clay Johnson <cjohnson@gradle.com> Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Signed-off-by: Matthew Haughton <3flex@users.noreply.github.com>
Fixed - I've also updated README in the last commit as there were a few references (and a link) to settings.gradle. Please check that before merging, thanks! |
Closes #186
The diff from GitHub makes it hard to review, but the commit history should be clear.