Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,24 @@ jobs:
JAVA_OPTS: "-Xmx8g -Dfile.encoding=UTF-8 -Djava.awt.headless=true -Dkotlin.daemon.jvm.options=-Xmx6g"

- name: Upload Reports
if: always()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argument for keeping it - there was a case when our tests that use JUnit5 weren't running because of the misconfiguraion, so we didn't know which tests are run and which are not.
Not a strong opinion so if there are any downsides in having it, let's switch to ${{ !cancelled() }} indeed.

if: ${{ !cancelled() }}
uses: actions/upload-artifact@v4
with:
name: reports
path: |
**/build/reports/

- name: Publish Test Report
uses: mikepenz/action-junit-report@v5
if: ${{ !cancelled() }} # always run even if the previous step fails
with:
report_paths: '**/test-results/**/TEST-*.xml'
detailed_summary: true
flaky_summary: true
include_empty_in_summary: false
include_time_in_summary: true
annotate_only: true

- name: Disable Auto-Merge on Fail
if: failure() && github.event_name == 'pull_request'
run: gh pr merge --disable-auto "$PR_URL"
Expand Down
Loading