ci: Cosmetics

Author: krlmlr

.github/workflows/commit/action.yml

@@ -22,6 +22,14 @@ runs:
         fi
       shell: bash
 
+    - name: Write diff and fail for workflow_dispatch
+      if: steps.check.outputs.has_changes == 'true' && github.event_name == 'workflow_dispatch'
+      run: |
+        echo "Changes detected in workflow_dispatch build. Diff:"
+        git diff
+        exit 1
+      shell: bash
+
     - name: Determine repository state
       if: steps.check.outputs.has_changes == 'true'
       id: repo-state

.github/workflows/copilot-setup-steps.yaml

@@ -6,10 +6,10 @@ on:
   workflow_dispatch:
   push:
     paths:
-      - .github/workflows/copilot-setup-steps.yml
+      - .github/workflows/copilot-setup-steps.yaml
   pull_request:
     paths:
-      - .github/workflows/copilot-setup-steps.yml
+      - .github/workflows/copilot-setup-steps.yaml
 
 jobs:
   # The job MUST be called `copilot-setup-steps` or it will not be picked up by Copilot.

.github/workflows/dep-suggests-matrix/action.R

@@ -15,7 +15,7 @@ get_deps <- function() {
 
   # Too big to fail, or can't be avoided:
   off_limits <- c("testthat", "rmarkdown", "rcmdcheck", deps_df_hard, deps_df_base)
-  off_limits_dep <- unlist(tools::package_dependencies(off_limits, db = installed.packages(), recursive = TRUE, which = "strong"))
+  off_limits_dep <- unlist(tools::package_dependencies(off_limits, recursive = TRUE, which = "strong"))
   setdiff(packages, c(off_limits, off_limits_dep))
 }