[static] ignore in todos (reverted)

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

Author: isegall-da

.github/workflows/pr_non_contributors.yml

@@ -2,8 +2,7 @@ name: CI on PRs from forks
 on:
   pull_request_target:
     types: [ opened, synchronize, reopened ]
-    branches: [ main, isegall/fix-fork-ci ] # Only run on PRs with `main` as their base
-    # FIXME
+    branches: [ main ] # Only run on PRs with `main` as their base
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.event.pull_request.head.ref }}

scripts/todo/src/checkTodos.sc

@@ -77,7 +77,8 @@ val fixedIssuesCurrentPR: Set[Int] = {
   prNumO.fold(Set.empty[Int])({ prNum =>
     val prInfo = Seq("hub", "pr", "show", "-f", "%b", prNum).!!
     val branch = Seq("hub", "pr", "show", "-f", "%H", prNum).!!.trim
-    val commits = Seq("git", "log", s"main..$branch").!!
+    // When running from forks, we don't currently check the commit messages on the branch
+    val commits = Try(Seq("git", "log", s"main..$branch").!!).getOrElse("")
     val issueCloseKeywords =
       Seq("close", "closes", "closed", "fix", "fixes", "fixed", "resolve", "resolves", "resolved")
         .map(w => s"($w)")