ibzib commented on pull request #14134:
URL: https://github.com/apache/beam/pull/14134#issuecomment-790801100


   > If I could figure out how to add all the mustRunAfter calls in the 
definition of the Precommit task so they were all in one place and only applied 
locally to that task, then I'd be ok with it.
   
   You'd have to separate the precommit task into subtasks:
   
   ```gradle
   task("goBuildAll") {
     dependsOn(":foo:goBuild")
     dependsOn(":bar:goBuild")
     // ...
   }
   
   task("goTestAll") {
     mustRunAfter(":goBuildAll")
     dependsOn(":sdks:go:goTest")
   }
   
   task("goPreCommit") {
     dependsOn(":goBuildAll")
     dependsOn(":goTestAll")
   }
   ```
   
   The main reason to fix this in Gradle and not the Jenkins test definitions 
is that Jenkins tests are a pain to run locally. I want to be able to run 
`./gradlew :goPreCommit` on my machine without it flaking.


----------------------------------------------------------------
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