Fix get-changed-files (#1)

* add test echo statements

* be more explicit in push run scenarios to avoid running on pushes to PR branches

.github/workflows/get-changed-files.yml

@@ -12,6 +12,7 @@ jobs:
       changed-files: ${{ toJSON(steps.changed-files.outputs) }}
     steps:
       - uses: actions/checkout@v4
+
       - name: Get Changed Files
         id: changed-files
         uses: dorny/paths-filter@v2
@@ -25,3 +26,9 @@ jobs:
             deleted:
               - deleted:
                 - '**'
+
+      - name: Echo GitHub Ref
+        run: echo "${{ github.ref }}  ${{ github.ref_name }}"
+
+      - name: Echo Changed Files Output
+        run: echo "${{ toJSON(steps.changed-files.outputs) }}"

.github/workflows/pr.yml

@@ -1,18 +1,20 @@
 name: Pull Request or Push
 
-# on: [push, pull_request]
 on:
-  workflow_call:
-    inputs:
-      changed-files:
-        required: true
-        type: string
-        description: JSON string containing information about changed files
+  push:
+    branches:
+      - 'main'  # Run on pushes to main even though we shouldn't do this
+    tags-ignore:
+      - '*'   # Ignore pushes to tags
+  pull_request:
 
 jobs:
-  ci:
-    name: CI
-    uses: salt-extensions/central-artifacts/.github/workflows/ci.yml@main
-    permissions:
-      contents: write
-      pull-requests: read
+  call-get-changed-files:
+    uses: salt-extensions/central-artifacts/.github/workflows/get-changed-files.yml@main
+
+  call-pre-commit-action:
+    uses: salt-extensions/central-artifacts/.github/workflows/pre-commit-action.yml@main
+    needs:
+      - call-get-changed-files
+    with:
+      changed-files: ${{ needs.call-get-changed-files.outputs.changed-files }}