fix: use arg file for publish script

hlxsites · Sep 14, 2023 · f2251ea · f2251ea
1 parent 2d9dc73
commit f2251ea
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 8 deletions.
diff --git a/.github/workflows/docs.yaml b/.github/workflows/docs.yaml
@@ -19,16 +19,19 @@ jobs:
       - name: Get changed files
         id: changedfiles
         run: |
-          EOF=$(dd if=/dev/urandom bs=15 count=1 status=none | base64)
-          echo "all<<$EOF" >> "$GITHUB_OUTPUT"
-          git diff --name-only --diff-filter=ACMRT ${{ github.event.before }} ${{ github.event.after }} | xargs >> "$GITHUB_OUTPUT"
-          echo "$EOF" >> "$GITHUB_OUTPUT"
-          cat "$GITHUB_OUTPUT"
+          ARG_FILE="${{ runner.temp }}/to_publish"
+          echo "arg file: $ARG_FILE"
+          #EOF=$(dd if=/dev/urandom bs=15 count=1 status=none | base64)
+          #echo "all<<$EOF" >> "$GITHUB_OUTPUT"
+          git diff --name-only --diff-filter=ACMRT ${{ github.event.before }} ${{ github.event.after }} | xargs > "$ARG_FILE"
+          #echo "$EOF" >> "$GITHUB_OUTPUT"
+          #cat "$GITHUB_OUTPUT"
+          cat "$ARG_FILE"
 
   publish:
     runs-on: ubuntu-latest
     needs: changedfiles
-    if: ${{ needs.changedfiles.outputs.all }}
+    # if: ${{ needs.changedfiles.outputs.file }}
     steps:
       - name: Checkout repository
         uses: actions/checkout@v3
@@ -37,4 +40,4 @@ jobs:
         with:
           node-version: '18'
       - run: npm ci
-      - run: npm run publish:batch ${{ needs.changedfiles.outputs.all }}
+      - run: npm run publish:batch -f ${{ runner.temp }}/to_publish
diff --git a/bin/batch-publish.js b/bin/batch-publish.js
@@ -74,7 +74,16 @@ function cleanPath(ppath) {
 }
 
 async function batchPublish(ppaths) {
-  const paths = ppaths.filter(isDocPath).map(cleanPath);
+  let paths = ppaths;
+
+  // get file as array, whitespace delimited
+  if (paths[0] === '-f' || paths[0] === '--file') {
+    const data = fs.readFileSync(paths[1]);
+    const text = data.toString('utf8');
+    paths = text.split(/\s+/);
+  }
+
+  paths = paths.filter(isDocPath).map(cleanPath);
   return processQueue(paths, publishDoc);
 }