emilymye commented on a change in pull request #13211:
URL: https://github.com/apache/beam/pull/13211#discussion_r521703844



##########
File path: sdks/java/container/build.gradle
##########
@@ -86,51 +76,21 @@ licenseReport {
   renderers = [new JsonReportRenderer()]
 }
 
-def imageJavaVersion = project.hasProperty('imageJavaVersion') ? 
project.findProperty('imageJavaVersion') : '8'
-docker {

Review comment:
       My current approach was just to just add a `task docker { throw ... }` 
block to replace the Docker plugin-created tasks, but but I can't figure out 
how to get the license report to ignore this task (and avoid throwing the 
error). This might just be a gradle question but a) should I be throwing the 
error differently and b) should the license report be the same task for both 
java8/11?




----------------------------------------------------------------
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:
[email protected]


Reply via email to