fix: use need syntax

siemens · Mar 20, 2024 · 2dd4fee · 2dd4fee
1 parent d31edf4
commit 2dd4fee
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 16 deletions.
diff --git a/.github/workflows/deploy-preview.yml b/.github/workflows/deploy-preview.yml
@@ -27,15 +27,16 @@ jobs:
       with:
         files_yaml: |
           core:
-            - 'packages/core/**'
+            - packages/core/**
           docs:
-            - 'packages/documentation/**'
-            - 'packages/html-test-app/**'
-            - 'packages/angular-test-app/**'
-            - 'packages/react-test-app/**'
-            - 'packages/vue-test-app/**'
+            - packages/documentation/**
+            - packages/html-test-app/**
+            - packages/angular-test-app/**
+            - packages/react-test-app/**
+            - packages/vue-test-app/**
 
   deploy-preview:
+    needs: changes
     if: ${{ needs.changes.outputs.core_any_changed == 'true' && needs.changes.outputs.docs_any_changed == 'true' }}
     runs-on: ubuntu-latest
     steps:

diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -27,17 +27,17 @@ jobs:
       with:
         files_yaml: |
           core:
-            - 'packages/core/**'
+            - packages/core/**
           angular:
-            - 'packages/angular/**'
+            - packages/angular/**
           aggrid:
-            - 'packages/aggrid/**'
+            - packages/aggrid/**
           docs:
-            - 'packages/documentation/**'
-            - 'packages/html-test-app/**'
-            - 'packages/angular-test-app/**'
-            - 'packages/react-test-app/**'
-            - 'packages/vue-test-app/**'
+            - packages/documentation/**
+            - packages/html-test-app/**
+            - packages/angular-test-app/**
+            - packages/react-test-app/**
+            - packages/vue-test-app/**
 
   build:
     needs: changes
@@ -82,7 +82,7 @@ jobs:
 
   visual-testing-aggrid:
     if: ${{ needs.changes.outputs.core_any_changed == 'true' || needs.changes.outputs.aggrid_any_changed == 'true' }}
-    needs: [build]
+    needs: [build, changes]
     timeout-minutes: 10
     runs-on: ubuntu-latest
     container:
@@ -128,7 +128,7 @@ jobs:
 
   unit:
     if: ${{ needs.changes.outputs.core_any_changed == 'true' || needs.changes.outputs.angular_any_changed == 'true' }}
-    needs: [build]
+    needs: [build, changes]
     timeout-minutes: 10
     runs-on: ubuntu-latest
     container: