Fix matrix def, markdown step, arts name

Signed-off-by: Michal Czyz <[email protected]>
antmicro · Mar 12, 2024 · 9032a88 · 9032a88
1 parent d65bd86
commit 9032a88
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 13 deletions.
diff --git a/.github/actions/implementation/action.yml b/.github/actions/implementation/action.yml
@@ -29,9 +29,7 @@ runs:
     - name: Summary page
       shell: bash
       run: |
-        echo "| Module  | ${{inputs.xls_module}} |" >> $GITHUB_STEP_SUMMARY
-        echo "| ------------- | ------------- |" >> $GITHUB_STEP_SUMMARY
-
+        echo "Module ${{inputs.xls_module}}" >> $GITHUB_STEP_SUMMARY
 
     - name: IR
       shell: bash
@@ -41,7 +39,7 @@ runs:
     - name: Summary page
       shell: bash
       run: |
-        echo "| Generate IR | :white_check_mark: |" >> $GITHUB_STEP_SUMMARY
+        echo "Generate IR :white_check_mark:" >> $GITHUB_STEP_SUMMARY
 
     - name: Verilog
       shell: bash
@@ -51,7 +49,7 @@ runs:
     - name: Summary page
       shell: bash
       run: |
-        echo "| Verilog codegen | :white_check_mark: |" >> $GITHUB_STEP_SUMMARY
+        echo "Verilog codegen | :white_check_mark:" >> $GITHUB_STEP_SUMMARY
 
     - name: Synthesis
       shell: bash
@@ -61,7 +59,7 @@ runs:
     - name: Summary page
       shell: bash
       run: |
-        echo "| Synthesis | :white_check_mark: |" >> $GITHUB_STEP_SUMMARY
+        echo -n "Synthesis | :white_check_mark:" >> $GITHUB_STEP_SUMMARY
 
     - name: P&R
       shell: bash
@@ -71,13 +69,22 @@ runs:
     - name: Summary page
       shell: bash
       run: |
-        echo "| Place & Route | :white_check_mark: |" >> $GITHUB_STEP_SUMMARY
+        echo -n "Place & Route | :white_check_mark:" >> $GITHUB_STEP_SUMMARY
+
+      # ${variable/character_to_replace/new_character}
+      # ${variable/ slash / underscore }
+    - name: Prepare artifact name
+      shell: bash
+      run: |
+        name_input=${{inputs.xls_module}}/${{inputs.rule_ir}}
+        name_output="${name_input//\//_}"
+        echo "{artifact_name}={${name_output}}" >> "$GITHUB_ENV"
 
     - name: Artifacts
       if: always()
       uses: actions/upload-artifact@v4
       with:
-        name: artifacts-impl-${{inputs.xls_module}}
+        name: artifacts-impl-${{ env.artifact_name }}
         path: |
           ./bazel-bin/${{inputs.xls_module}}/*.log
           ./bazel-bin/${{inputs.xls_module}}/*.textproto

diff --git a/.github/workflows/xls-modules.yml b/.github/workflows/xls-modules.yml
@@ -77,11 +77,11 @@ jobs:
         id: implementation
         uses: ./.github/actions/implementation
         with:
-          xls_module: "xls/modules/dma"
-          rule_ir: "csr_opt_ir_benchmark"
-          rule_verilog: "verilog_csr"
-          rule_synthesis: "csr_benchmark_synth"
-          rule_pnr: "csr_place_and_route"
+          xls_module: ${{ matrix.module.xls_module }}
+          rule_ir: ${{ matrix.module.rule_ir }}
+          rule_verilog: ${{ matrix.module.rule_verilog }}
+          rule_synthesis: ${{ matrix.module.rule_synthesis }}
+          rule_pnr: ${{ matrix.module.rule_pnr }}
 
   test:
     needs: build