-
Notifications
You must be signed in to change notification settings - Fork 19
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
Integrate buildconfig plugin to project to replace old version templates work #677
Conversation
The old API is now deprecated
…ties and Kotlin file.
This commit renames the agp-handler-80 directory to agp-handler-82. Additionally, it updates the references to agp-handler-80 to now point to agp-handler-82 in the settings.gradle.kts file and the slack-plugin/build.gradle.kts file. This ensures that the correct version of the agp-handler is being used in the project.
@@ -1,9 +0,0 @@ | |||
@file:JvmName("KotlinBuildConfig") |
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.
Does this reference still work with this file removed?
https://github.com/slackhq/slack-gradle-plugin/pull/677/files#diff-c0dfa6bc7a8685217f70a860145fbdf416d449eaff052fa28352c5cec1a98c06R180.
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.
That's different than this file actually, they're just named the same. That other def is here: https://github.com/slackhq/slack-gradle-plugin/pull/677/files#diff-c0dfa6bc7a8685217f70a860145fbdf416d449eaff052fa28352c5cec1a98c06R122
project.pluginManager.withPlugin("com.github.gmazzo.buildconfig") { | ||
configure<BuildConfigExtension> { | ||
buildConfigField("String", "KOTLIN_VERSION", "\"$kotlinVersion\"") | ||
// Using Any here due to https://github.com/gmazzo/gradle-buildconfig-plugin/issues/9 |
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.
PR'd a hopeful fix here gmazzo/gradle-buildconfig-plugin#98
This uses a nicer plugin to handle buildconfig constants