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

Upgrade to Gradle 6.7.1, shadow 6.1.0, jfrog build info extractor 4.18.3 #251

Merged
merged 2 commits into from
Mar 23, 2021

Conversation

martinda
Copy link
Contributor

@martinda martinda commented Jan 1, 2021

Proposing the following upgrades:

  • Gradle 6.7.1:
    • Upgrade to the plugins { ... } form of declaring plugins
    • Upgrade the configuration names for Gradle 6.7.1
    • Removed the maven plugin (maven-publish replaces it)
    • Upgrade the deprecated version attribute to the new archiveVersion in the jar attributes as per the gradle documentation
    • Upgrade the gradle wrapper
  • Shadow 6.1.0:
    • Upgrade com.github.jengelman.gradle.plugins:shadow:2.0.2 to id 'com.github.johnrengelman.shadow' version '6.1.0'
    • Remove shadowJar artifact as it is automatically added since version 6.1.0 PR #357
  • JFrog build info extractor 4.18.3 (trivial)

@cdancy cdancy self-assigned this Jan 8, 2021
@@ -18,7 +18,6 @@ publishing {
artifact sourcesJar
artifact testsJar
artifact javadocJar
artifact shadowJar
Copy link
Owner

Choose a reason for hiding this comment

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

@martinda why did we remove this? Will the shadow jar still got published if we remove it?

And apologies for the late responses ... just getting back into the swing of things now from the holidays.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hello @cdancy . It is automatically added since version shadow version 6.1.0 [PR #357]

@cdancy cdancy added this to the v2.7.2 milestone Jan 15, 2021
@cdancy cdancy merged commit b4f2385 into cdancy:master Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants