Skip to content

Commit

Permalink
Disable module metadata for all release artifacts (#3836)
Browse files Browse the repository at this point in the history
  • Loading branch information
heyams authored Aug 12, 2024
1 parent 56b59b8 commit ec143c4
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 12 deletions.
7 changes: 0 additions & 7 deletions agent/agent/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -165,13 +165,6 @@ tasks {
named("generateLicenseReport").configure {
dependsOn(cleanLicenses)
}

// Because we reconfigure publishing to only include the shadow jar, the Gradle metadata is not correct.
// Since we are fully bundled and have no dependencies, Gradle metadata wouldn't provide any advantage over
// the POM anyways so in practice we shouldn't be losing anything.
withType<GenerateModuleMetadata>().configureEach {
enabled = false
}
}

// Don't publish non-shadowed jar (shadowJar is in shadowRuntimeElements)
Expand Down
5 changes: 0 additions & 5 deletions agent/runtime-attach/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,4 @@ tasks {
attributes("Automatic-Module-Name" to "com.microsoft.applicationinsights.attach")
}
}

// disabling the publication of Gradle Module Metadata
withType<GenerateModuleMetadata>().configureEach {
enabled = false
}
}
8 changes: 8 additions & 0 deletions buildSrc/src/main/kotlin/ai.java-conventions.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,14 @@ tasks {
isPreserveFileTimestamps = false
isReproducibleFileOrder = true
}

// disabling the publication of Gradle Module Metadata
// Because we reconfigure publishing to only include the shadow jar, the Gradle metadata is not correct.
// Since we are fully bundled and have no dependencies, Gradle metadata wouldn't provide any advantage over
// the POM anyways so in practice we shouldn't be losing anything.
withType<GenerateModuleMetadata>().configureEach {
enabled = false
}
}

normalization {
Expand Down

0 comments on commit ec143c4

Please sign in to comment.