[GitHub] [yetus] ndimiduk commented on issue #86: YETUS-934. Precommit does not accept a caller's value of MAVEN_ARGS

[GitBox <gi...@apache.org>]

ndimiduk commented on issue #86: YETUS-934. Precommit does not accept a caller's value of MAVEN_ARGS
URL: https://github.com/apache/yetus/pull/86#issuecomment-583488943
 
 
   > Since the goal here is to set options for surefire
   
   Heya @aw-was-here. This is not quite correct. We're trying to add the `--threads ...` argument to the maven incantation. This is not for surefire's benefit, but for maven itself. Either way, we're just adding another argument to the command, so maybe it doesn't matter either way? I don't know if maven expects its optional arguments to come ahead of the positional arguments.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services