From 193c5081428b4fd3dfda196bc56e53278ce1db75 Mon Sep 17 00:00:00 2001
From: Ethan Jacob Marx <ethan.marx@ligo.org>
Date: Sat, 25 May 2024 13:22:39 -0700
Subject: [PATCH] get project tests running

---
 .github/workflows/project-tests.yaml | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/project-tests.yaml b/.github/workflows/project-tests.yaml
index 86507d52..e341dc06 100644
--- a/.github/workflows/project-tests.yaml
+++ b/.github/workflows/project-tests.yaml
@@ -2,7 +2,7 @@ name: project tests
 
 on:
   pull_request:
-   types:
+    types:
       - opened
       - reopened
       - synchronize
@@ -17,27 +17,24 @@ jobs:
     outputs:
       projects: ${{ steps.filter.outputs.projectchanges }}
     steps:
-    - uses: actions/checkout@v2
-    - uses: dorny/paths-filter@v2
+    - name: Checkout repository
+      uses: actions/checkout@v2
+    - name: Filter project changes
       id: filter 
+      uses: dorny/paths-filter@v2
       with:
         filters: .github/project-filters.yaml
-    if: github.event.pull_request.draft == false
+      if: github.event.pull_request.draft == false
 
-  # second job takes those outputs and runs
-  # unit tests on these libs and projects
   tests:
     runs-on: ubuntu-latest
     needs: projectchanges
     strategy:
-      fail-fast: false # is this desired behavior here?
+      fail-fast: false
       matrix:
         project: ${{ fromJSON(needs.projectchanges.outputs.projects) }}
         exclude:
-          # don't run non-existent .github/workflow tests,
-          # and exclude projects, whose tests are run in 
-          # a different CI script
-          - project: 'workflow'
+          - project: 'workflow' # Fixed the exclusion key
     permissions:
       packages: read
     container: