-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Upgrade com.github.johnrengelman:shadow to 8.1.1 #3255
Conversation
bac7ff2
to
2243ec3
Compare
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.
Looks like we lost the 4 space indent, these new changes are 2 indent.
build.gradle.kts
Outdated
maven { | ||
url = uri("https://plugins.gradle.org/m2/") | ||
} |
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.
Isn't this the same as gradlePluginPortal
?
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.
This is how it was written at https://plugins.gradle.org/plugin/com.github.johnrengelman.shadow
} | ||
|
||
apply(plugin = "com.github.johnrengelman.shadow") |
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.
Do you need to apply this? If it was attached to build script classpath its already there for building.
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.
again, this is how it was at https://plugins.gradle.org/plugin/com.github.johnrengelman.shadow
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.
Looks like that says two different methods though and you have both, so just a bit lost.
@Phone-Guru - Thanks for spotting this out of date version, but I went ahead and did what I was trying to explain. Its tough to follow online docs because they don't really fully show all the ways a plugin works. They don't show buildscript vs runtime dependencies and suffer in groovy vs kotlin. So we wouldn't really go anywhere just taking the examples online. Unfortunately Gradle has a mix of a lot of old/new docs around. So take a look at this diff if you are curious what I did - #3259 |
No description provided.