Merge pull request #2417 from consideRatio/pr/second-attempt-on-fixing-set-output

ci: attempt to fix set-output again

Author: manics

.github/workflows/watch-dependencies.yaml

@@ -76,15 +76,16 @@ jobs:
         if: steps.local.outputs.tag != steps.latest.outputs.tag
         run: git --no-pager diff --color=always
 
-      - name: PR summary
+      - name: Fetch PR summary
         id: prsummary
         if: steps.local.outputs.tag != steps.latest.outputs.tag
-        # Needs to be a single line string
-        # https://trstringer.com/github-actions-multiline-strings/#option-1---string-substitution
         run: |
           pip install PyGithub
-          PR_LIST=$(./scripts/get-prs.py ${{matrix.repository}} ${{steps.local.outputs.tag}} ${{steps.latest.outputs.tag}} --github-action-escape)
-          echo "prs=$PR_LIST" >> $GITHUB_OUTPUT
+          ./scripts/get-prs.py \
+              ${{matrix.github_repo}} \
+              ${{steps.local.outputs.version}} \
+              ${{steps.latest.outputs.version}} \
+              --write-github-actions-output=prs
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
@@ -204,15 +205,16 @@ jobs:
       - name: git diff
         run: git --no-pager diff --color=always
 
-      - name: PR summary
+      - name: Fetch PR summary
         id: prsummary
         if: matrix.github_repo && (steps.local.outputs.version != steps.latest.outputs.version)
-        # Needs to be a single line string
-        # https://trstringer.com/github-actions-multiline-strings/#option-1---string-substitution
         run: |
           pip install PyGithub
-          PR_LIST=$(./scripts/get-prs.py ${{matrix.github_repo}} ${{steps.local.outputs.version}} ${{steps.latest.outputs.version}} --github-action-escape)
-          echo "prs=$PR_LIST" >> $GITHUB_OUTPUT
+          ./scripts/get-prs.py \
+              ${{matrix.github_repo}} \
+              ${{steps.local.outputs.version}} \
+              ${{steps.latest.outputs.version}} \
+              --write-github-actions-output=prs
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 

scripts/get-prs.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python
 import os
 import re
+import uuid
 from argparse import ArgumentParser
 
 import github
@@ -30,9 +31,8 @@ def extract_gitref(s):
 parser.add_argument("start", help="commit or image/chart version from which to start")
 parser.add_argument("end", help="commit or image/chart version to which to end")
 parser.add_argument(
-    "--github-action-escape",
-    action="store_true",
-    help="Escape output for GitHub Action",
+    "--write-github-actions-output",
+    help="Name of a GitHub Action's output variable to write to",
 )
 parser.add_argument(
     "--max-commits",
@@ -65,7 +65,16 @@ def extract_gitref(s):
     ]
 
 md = ["# PRs"] + pr_summaries + ["", f"{r.html_url}/compare/{start}...{end}"]
-if args.github_action_escape:
-    print("%0A".join(md))
+md = "\n".join(md)
+
+if args.write_github_actions_output:
+    # GitHub Actions docs on setting a output variable with a multiline string:
+    # https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#multiline-strings
+    #
+    eof_marker = str(uuid.uuid4()).replace("-", "_")
+    with open(os.environ["GITHUB_OUTPUT"], "a") as f:
+        print(f"{args.write_github_actions_output}<<{eof_marker}", file=f)
+        print(md, file=f)
+        print(eof_marker, file=f)
 else:
-    print("\n".join(md))
+    print(md)