Skip to content

Remove merge-multiple option from test reports download#6

Merged
TheMeinerLP merged 1 commit into
mainfrom
claude/implement-release-please-9inVJ
May 11, 2026
Merged

Remove merge-multiple option from test reports download#6
TheMeinerLP merged 1 commit into
mainfrom
claude/implement-release-please-9inVJ

Conversation

@TheMeinerLP
Copy link
Copy Markdown
Contributor

Summary

Removed the merge-multiple: true option from the test reports artifact download step in the GitHub Actions workflow.

Changes

  • Removed the merge-multiple: true configuration from the "Download all test reports" step in the gradle-build-pr.yml workflow

Details

This change simplifies the artifact download configuration by removing the merge-multiple option. This may indicate either that the option is no longer needed for the current workflow requirements, or that artifacts should be kept separate rather than merged during download.

https://claude.ai/code/session_01DbyhbsRLEJ2EfGRppFnDY6

…L collisions

Removing merge-multiple: true so each matrix runner's artifact extracts
into its own subdirectory (artifacts/test-reports-<os>-jdk<ver>/...).
Previously, all matrix OSes wrote the same path
common/build/test-results/test/TEST-*.xml; concurrent extraction with
merge-multiple could leave concatenated XML, causing
EnricoMi/publish-unit-test-result-action to fail with
"Extra content at the end of the document".

The junit_files glob still picks up every matrix runner's reports.
@TheMeinerLP TheMeinerLP merged commit 168ca0a into main May 11, 2026
1 check passed
@TheMeinerLP TheMeinerLP deleted the claude/implement-release-please-9inVJ branch May 11, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant