Skip to content

Commit

Permalink
Refs #20400: Use test step conclusion to decide whether to run test r…
Browse files Browse the repository at this point in the history
…eport step

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
  • Loading branch information
EduPonz committed Feb 9, 2024
1 parent d13d9ad commit a0fb86a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/reusable-windows-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ jobs:

- name: Test Report
uses: eProsima/eProsima-CI/external/test-reporter@main
if: (success() || failure()) && ${{ !contains(github.event.pull_request.labels.*.name, 'no-test') }}
if: ${{ (steps.test.conclusion == 'success' || steps.test.conclusion == 'failure') && !contains(github.event.pull_request.labels.*.name, 'no-test') }}
with:
name: "Report: ${{ matrix.cmake-config }} | ${{ matrix.vs-toolset }} "
path: "${{ steps.test.outputs.ctest_results_path }}*.xml"
Expand Down

0 comments on commit a0fb86a

Please sign in to comment.