Develop

[ptr727]

No description provided.

[Copilot]

Pull Request Overview

This PR updates the CI workflow by removing an unnecessary dependency for the matrix setup and adding a new job to explicitly check the results of key jobs due to GitHub Actions limitations.

• Removed needs: unittests from the setmatrix job so the matrix is always generated.
• Added a check-workflow-status job to fail the workflow if unittests or builddocker jobs fail or are cancelled.
• Included a TODO comment linking to GitHub’s discussion on selecting multiple dependencies.

Comments suppressed due to low confidence (3)

.github/workflows/TestPullRequest.yml:133

• The first argument "check-labels" does not match any defined job. It should reference the actual job name (e.g., "unittests") to correctly report its status.

          exit_on_result "check-labels" "${{ needs.unittests.result }}"

.github/workflows/TestPullRequest.yml:134

• The label "run-tests" here doesn't correspond to any job in needs. Consider using "builddocker" or updating the job reference to match.

          exit_on_result "run-tests" "${{ needs.builddocker.result }}"

.github/workflows/TestPullRequest.yml:119

• [nitpick] The inline array syntax with trailing commas is inconsistent with the rest of the workflow. Consider using the standard block list (`- unittests

• builddocker`) for readability and consistency.

      [