Proper inputs

Author: ychin

.github/actions/test_artifacts/action.yml

@@ -1,31 +1,42 @@
 name: 'test_artifacts'
 description: "Upload failed test artifacts"
+inputs:
+  artifact-name:
+    description: Name of the artifact
+    required: true
+
 runs:
   using: "composite"
   steps:
-    - name: Collect matrix properties for naming
-      uses: actions/github-script@v8
-      id: matrix-props
-      env:
-        MATRIX_PROPS: ${{ toJSON(inputs) }}
-      with:
-        # An array-flattening-to-string JavaScript function.
-        script: |
-          const f = function (x) { return x.toString().length > 0; }
-          const g = function (x) {
-            return (Array.isArray(x))
-              ? x.filter(f)
-                .map((function (h) { return function (y) { return h(y); }; })(g))
-                .join('-')
-              : x;
-          }
-          return Object.values(JSON.parse(process.env.MATRIX_PROPS))
-            .filter(f)
-            .map(g)
-            .join('-');
-        # By default, the JSON-encoded return value of the function is
-        # set as the "result".
-        result-encoding: string
+# MacVim: We don't use a matrix within the reused
+# workflow, and so would prefer to manually pass
+# in the name of the artifact rather than deriving
+# it from the matrix automatically like in Vim
+# upstream.
+
+#    - name: Collect matrix properties for naming
+#      uses: actions/github-script@v8
+#      id: matrix-props
+#      env:
+#        MATRIX_PROPS: ${{ toJSON(inputs) }}
+#      with:
+#        # An array-flattening-to-string JavaScript function.
+#        script: |
+#          const f = function (x) { return x.toString().length > 0; }
+#          const g = function (x) {
+#            return (Array.isArray(x))
+#              ? x.filter(f)
+#                .map((function (h) { return function (y) { return h(y); }; })(g))
+#                .join('-')
+#              : x;
+#          }
+#          return Object.values(JSON.parse(process.env.MATRIX_PROPS))
+#            .filter(f)
+#            .map(g)
+#            .join('-');
+#        # By default, the JSON-encoded return value of the function is
+#        # set as the "result".
+#        result-encoding: string
     - name: Upload failed tests
       uses: actions/upload-artifact@v4
       with:
@@ -35,7 +46,7 @@ runs:
           github.run_attempt,
           github.job,
           strategy.job-index,
-          inputs.os) }}
+          inputs.artifact-name) }}
 
         # A file, directory or wildcard pattern that describes what
         # to upload.

.github/workflows/macvim-buildtest.yaml

@@ -385,6 +385,8 @@ jobs:
       - name: Upload failed test files
         #if: ${{ !cancelled() && failure() }}
         uses: ./.github/actions/test_artifacts
+        with:
+          artifact-name: ${{ inputs.os }}
 
       - name: Build MacVim dmg image
         if: inputs.publish && (startsWith(github.ref, 'refs/tags/') || github.ref == 'refs/heads/master')