Skip to content

Commit

Permalink
Upload tests result artifacts on Windows
Browse files Browse the repository at this point in the history
  • Loading branch information
tvallin committed Jan 16, 2025
1 parent a756770 commit 294bda2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
7 changes: 3 additions & 4 deletions .github/actions/common/action.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# Copyright (c) 2023, 2024 Oracle and/or its affiliates.
# Copyright (c) 2023, 2025 Oracle and/or its affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -57,7 +57,7 @@ inputs:
required: false
default: ''
test-artifact-name:
description: Name of the test artifact to create (excluded on windows), if non empty tests are archived
description: Name of the test artifact to create, if non empty tests are archived
required: false
default: ''
free-space:
Expand Down Expand Up @@ -157,8 +157,7 @@ runs:
run: ${{ inputs.run }}
shell: bash
- name: Archive test results
# https://github.com/actions/upload-artifact/issues/240
if: ${{ inputs.test-artifact-name != '' && runner.os != 'Windows' && always() }}
if: ${{ inputs.test-artifact-name != '' && always() }}
uses: actions/upload-artifact@v4
with:
if-no-files-found: 'ignore'
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/validate.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# Copyright (c) 2023, 2024 Oracle and/or its affiliates.
# Copyright (c) 2023, 2025 Oracle and/or its affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -115,7 +115,7 @@ jobs:
- uses: ./.github/actions/common
with:
build-cache: read-only
test-artifact-name: tests-${{ matrix.moduleSet }}
test-artifact-name: tests-${{ matrix.moduleSet }}-${{ matrix.os }}
run: |
mvn ${MAVEN_ARGS} \
-DreactorRule=tests \
Expand Down

0 comments on commit 294bda2

Please sign in to comment.