ci: review required jobs (#9124)

* ci: review required jobs

* retrigger

* rm trigger

.github/workflows/ci_metrics.yml

@@ -4,7 +4,6 @@ on:
   workflow_run:
     workflows:
       - "end-to-end"
-      - "Linting"
       - "Tests"
     types:
       - completed

.github/workflows/tests.yml

@@ -482,6 +482,8 @@ jobs:
     # This way we'll be able to mark only this test as required
     # and skip it accordingly.
     needs:
+      - check-imports
+      - lint
       - integration-tests-linux
       - integration-tests-macos
       - integration-tests-windows

.github/workflows/tests_skipper_trigger.yml

@@ -11,7 +11,7 @@ on:
     paths-ignore:
       # we skip the tests unless the code changes. The problem is that GitHub will run the check anyway if any other
       # file outside the code changed (e.g. the release notes). Hence, we need a second filter down below.
-      # keep the list in sync with the paths defined in the `tests.yml` workflow
+      # keep the list in sync with the paths defined in the `tests.yml` workflow and below in this file
       - "haystack/**/*.py"
       - "haystack/core/pipeline/predefined/*"
       - "test/**/*.py"
@@ -32,13 +32,14 @@ jobs:
         id: changes
         uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36
         with:
-          # keep the list in sync with the paths defined in the `tests.yml` workflow
+          # keep the list in sync with the paths defined in the `tests.yml` workflow and above in this file
           filters: |
             code_changes:
-              - haystack/**/*.py
-              - "haystack/templates/predefined/*"
-              - test/**/*.py
+              - "haystack/**/*.py"
+              - "haystack/core/pipeline/predefined/*"
+              - "test/**/*.py"
               - "pyproject.toml"
+              - ".github/utils/check_imports.py"
 
   trigger-catch-all:
     name: Tests completed