[
https://issues.apache.org/jira/browse/BEAM-7719?focusedWorklogId=275093&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-275093
]
ASF GitHub Bot logged work on BEAM-7719:
----------------------------------------
Author: ASF GitHub Bot
Created on: 10/Jul/19 22:20
Start Date: 10/Jul/19 22:20
Worklog Time Spent: 10m
Work Description: lukecwik commented on pull request #9036: [BEAM-7719]
Ensure that publishing vendored artifacts checks the contents of the jar before
publishing.
URL: https://github.com/apache/beam/pull/9036#discussion_r302295638
##########
File path:
buildSrc/src/main/groovy/org/apache/beam/gradle/VendorJavaPlugin.groovy
##########
@@ -146,6 +146,8 @@ artifactId=${project.name}
// Only publish vendored dependencies if specifically requested.
if (project.hasProperty("vendoredDependenciesOnly")) {
project.apply plugin: 'maven-publish'
+ // Ensure that we validate the contents of the jar before publishing
+ project.publish.dependsOn project.check
Review comment:
It will only run check on this specific project which are projects which
perform vendoring and apply this module so only projects under `vendor/...`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 275093)
Time Spent: 1h (was: 50m)
> Ensure that publishing vendored artifacts first validates there contents
> ------------------------------------------------------------------------
>
> Key: BEAM-7719
> URL: https://issues.apache.org/jira/browse/BEAM-7719
> Project: Beam
> Issue Type: Improvement
> Components: build-system
> Reporter: Luke Cwik
> Assignee: Luke Cwik
> Priority: Minor
> Time Spent: 1h
> Remaining Estimate: 0h
>
> During the release of vendored guava 26.0, it was discovered that we don't
> check the contents of the jars automatically.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
