Merge pull request #572 from Remi-Gau/env_var

[FIX] use local var for software name in bootstrap workflow
ReproNim · Sep 3, 2023 · da4f22c · da4f22c
2 parents 5da89b8 + 6de8619
commit da4f22c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 16 deletions.
diff --git a/.github/workflows/bootstrap.yml b/.github/workflows/bootstrap.yml
@@ -52,19 +52,6 @@ jobs:
     runs-on: ubuntu-22.04
 
     steps:
-      - name: check workflow inputs
-        run: |
-          software_name=${{ inputs.software_name }}
-          if [ -z "$software_name" ]; then
-            software_name="all"
-          fi
-          if [ "$software_name" = "all" ]; then
-            echo "testing all software"
-          else
-            echo "testing ${software_name}"
-          fi
-          export SOFTWARE_NAME=${software_name}
-
       - uses: actions/checkout@v3
         with:
           token: ${{ secrets.CI_FLOW }}
@@ -79,8 +66,17 @@ jobs:
 
       - name: Create workflows
         run: |
+          software_name=${{ inputs.software_name }}
+          if [ -z "$software_name" ]; then
+            software_name="all"
+          fi
+          if [ "$software_name" = "all" ]; then
+            echo "testing all software"
+          else
+            echo "testing ${software_name}"
+          fi
           git checkout -b test_docker_build
-          python .github/workflows/create_workflows.py ${SOFTWARE_NAME}
+          python .github/workflows/create_workflows.py --software_name ${software_name}
           ls -l .github/workflows
           git add .
           git config --global user.email "no-reply@repronim.org"

diff --git a/.github/workflows/create_workflows.py b/.github/workflows/create_workflows.py
@@ -207,12 +207,12 @@ def main(software_name="all"):
     choices.append("all")
 
     parser.add_argument(
-        "--software",
+        "--software_name",
         required=False,
         default="all",
         choices=choices,
         nargs=1,
     )
     args = parser.parse_args()
 
-    main(software_name=args.software[0])
+    main(software_name=args.software_name[0])