This is an automated email from the ASF dual-hosted git repository.

damccorm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new f93f62e9d82 reorder args to avoid newline issues (#28994)
f93f62e9d82 is described below

commit f93f62e9d82701a5a1c77dfc48723b74c15b9ca2
Author: Danny McCormick <[email protected]>
AuthorDate: Fri Oct 13 16:34:12 2023 -0400

    reorder args to avoid newline issues (#28994)
    
    * reorder args to avoid newline issues
    
    * Inline default args
---
 .github/actions/gradle-command-self-hosted-action/action.yml | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/.github/actions/gradle-command-self-hosted-action/action.yml 
b/.github/actions/gradle-command-self-hosted-action/action.yml
index cb793714aa9..906b35169d9 100644
--- a/.github/actions/gradle-command-self-hosted-action/action.yml
+++ b/.github/actions/gradle-command-self-hosted-action/action.yml
@@ -24,12 +24,6 @@ inputs:
     required: false
     description: 'Gradle options'
     default: ''
-  default-arguments:
-    required: false
-    description: 'Default gradle switches' # Copied from 
CommonJobProperties.groovy'
-    default: |
-      --continue -Dorg.gradle.jvmargs=-Xms2g -Dorg.gradle.jvmargs=-Xmx6g \
-      -Dorg.gradle.vfs.watch=false -Pdocker-pull-licenses
   max-workers:
     required: false
     description: 'Max number of workers'
@@ -47,5 +41,6 @@ runs:
         if [ -f ~/.m2/settings.xml ]; then
           rm ~/.m2/settings.xml
         fi
-        ./gradlew ${{ inputs.gradle-command }} --max-workers=${{ 
inputs.max-workers }} ${{ inputs.arguments }} \
-        ${{ inputs.default-arguments }}
\ No newline at end of file
+        ./gradlew ${{ inputs.gradle-command }} --max-workers=${{ 
inputs.max-workers }} --continue \
+         -Dorg.gradle.jvmargs=-Xms2g -Dorg.gradle.jvmargs=-Xmx6g 
-Dorg.gradle.vfs.watch=false -Pdocker-pull-licenses \
+        ${{ inputs.arguments }}

Reply via email to