Update results job to check skip_workflow on all declared dependencies (#13353)

* Initial plan

* Update results job to check skip_workflow on all declared dependencies

Co-authored-by: captainsafia <[email protected]>

* Add comment explaining skip_workflow check behavior for all dependencies

Co-authored-by: captainsafia <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: captainsafia <[email protected]>

Author: Copilot

.github/workflows/ci.yml

@@ -86,13 +86,18 @@ jobs:
     steps:
       - name: Fail if any of the dependent jobs failed
         # Don't fail if the workflow is being skipped.
+        # Check skip_workflow on all declared dependencies. Workflows without
+        # skip_workflow outputs (e.g., tests, build_cli_archives) evaluate to
+        # empty string, so the check still works ('!= true' is true for empty).
         #
         # For others 'skipped' can be when a transitive dependency fails and the dependent job gets
         # 'skipped'. For example, one of setup_* jobs failing and the Integration test jobs getting
         # 'skipped'
         if: >-
           ${{ always() &&
               needs.prepare_for_ci.outputs.skip_workflow != 'true' &&
+              needs.tests.outputs.skip_workflow != 'true' &&
+              needs.build_cli_archives.outputs.skip_workflow != 'true' &&
               (contains(needs.*.result, 'failure') ||
                contains(needs.*.result, 'cancelled') ||
                contains(needs.*.result, 'skipped')) }}