Repo Update

- A few misc changes and enhancements

Author: jarroddavis68

.github/workflows/push-to-discussions.yml

@@ -2,16 +2,15 @@ name: Push → Discussion
 
 on:
   push:
-    branches:
-      - '**'   # change to [ main ] if you only want main pushes
+    branches: ['**']
+  workflow_dispatch: {}   # allows manual run for testing
 
 permissions:
   contents: read
-  discussions: write   # needed to create discussions
+  discussions: write
 
 env:
-  # Must match an existing Discussions category in this repo
-  DISCUSSIONS_CATEGORY: Push Updates
+  DISCUSSIONS_CATEGORY: Push Updates   # must match the category name in Discussions
 
 jobs:
   post:
@@ -20,52 +19,56 @@ jobs:
       - name: Create a discussion for this push
         uses: actions/github-script@v7
         with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
           script: |
-            const core = require("@actions/core");
             const { owner, repo } = context.repo;
-            const push = context.payload;
-
-            // --- 1) Find category by name ---
             const desiredName = (process.env.DISCUSSIONS_CATEGORY || "").trim();
-            const { data: categories } = await github.request(
-              "GET /repos/{owner}/{repo}/discussions/categories",
-              { owner, repo }
-            );
-            if (!categories?.length) {
-              core.setFailed("No Discussions categories found. Enable Discussions and create at least one category.");
+
+            // 1) Fetch categories
+            let categories;
+            try {
+              const res = await github.request(
+                "GET /repos/{owner}/{repo}/discussions/categories",
+                { owner, repo }
+              );
+              categories = res.data || [];
+            } catch (e) {
+              core.setFailed(`Failed to fetch categories: ${e.message}`);
               return;
             }
-            const category = categories.find(c => c.name.toLowerCase() === desiredName.toLowerCase()) || categories[0];
 
-            // --- 2) Build title/body from push payload ---
-            const branch = (push.ref || "").replace("refs/heads/", "");
-            const pusher =
-              push.pusher?.name ||
-              push.sender?.login ||
-              "unknown";
+            // 2) Find category
+            const category = categories.find(
+              c => c.name.toLowerCase() === desiredName.toLowerCase()
+            );
+            if (!category) {
+              const names = categories.map(c => `- ${c.name} (id: ${c.id})`).join("\n");
+              core.setFailed(
+                `Category "${desiredName}" not found.\nAvailable categories:\n${names}`
+              );
+              return;
+            }
 
+            // 3) Build discussion content
+            const push = context.payload;
+            const branch = (push.ref || "").replace("refs/heads/", "") || "(unknown)";
+            const pusher = push.pusher?.name || push.sender?.login || "unknown";
             const commits = push.commits || [];
             const commitCount = commits.length;
 
-            // Title like: "Push to main by Jarrod — 3 commits"
-            const title = `Push to ${branch || "(unknown branch)"} by ${pusher} — ${commitCount} commit${commitCount === 1 ? "" : "s"}`;
+            const title = `Push to ${branch} by ${pusher} — ${commitCount} commit${commitCount === 1 ? "" : "s"}`;
 
-            // Each commit: short sha, first-line message, author, link
-            const lines = [];
-            for (const c of commits) {
-              const sha7 = (c.id || "").substring(0, 7);
-              const firstLine = (c.message || "").split("\n")[0];
+            const lines = commits.map(c => {
+              const sha7 = (c.id || "").slice(0,7);
+              const first = (c.message || "").split("\n")[0];
               const author = c.author?.name || "unknown";
-              const url = c.url || `https://github.com/${owner}/${repo}/commit/${c.id}`;
-              lines.push(`- [\`${sha7}\`](${url}) ${firstLine} — _${author}_`);
-            }
+              return `- [\`${sha7}\`](${c.url}) ${first} — _${author}_`;
+            });
 
-            // Compare URL (payload usually provides it)
             const compareUrl =
               push.compare ||
               `https://github.com/${owner}/${repo}/compare/${push.before}...${push.after}`;
 
-            // Top banner + details + footer
             const body = [
               `> _This discussion was created automatically by GitHub Actions from a \`push\` event._`,
               "",
@@ -79,16 +82,13 @@ jobs:
               "<sub>— end of automated log —</sub>"
             ].join("\n");
 
-            // --- 3) Create the Discussion ---
-            const { data: discussion } = await github.request(
-              "POST /repos/{owner}/{repo}/discussions",
-              {
-                owner,
-                repo,
-                title,
-                body,
-                category_id: category.id
-              }
-            );
-
-            core.info(`Created discussion: ${discussion.html_url}`);
+            // 4) Create the discussion
+            try {
+              const { data: discussion } = await github.request(
+                "POST /repos/{owner}/{repo}/discussions",
+                { owner, repo, title, body, category_id: category.id }
+              );
+              core.info(`Created discussion: ${discussion.html_url}`);
+            } catch (e) {
+              core.setFailed(`Failed to create discussion: ${e.message}`);
+            }

src/tests/libLLVM.Test.CodeGen.pas

@@ -192,7 +192,6 @@ class procedure TTestCodeGen.TestBuildAndLink();
   if not LStdErr.IsEmpty then
     TLLUtils.PrintLn(LStdErr);
 
-
   // === Results ===
   TLLUtils.PrintLn('LLD rc=%d canRunAgain=%s', [LRC, BoolToStr(LCan, True)]);
   TLLUtils.PrintLn('LL file:  %s', [LLLFile]);