Skip to content

[WIP] Switch XML format to JUnit for PHPUnit output#14

Draft
Claude wants to merge 3 commits into
mainfrom
claude/switch-xml-format-to-junit
Draft

[WIP] Switch XML format to JUnit for PHPUnit output#14
Claude wants to merge 3 commits into
mainfrom
claude/switch-xml-format-to-junit

Conversation

@Claude
Copy link
Copy Markdown

@Claude Claude AI commented May 25, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>switch xml format to junit</issue_title>
<issue_description>switch xml format to junit (from the original phpunit xml in directory). I want it to be just single file
it is produced by phpunit with param --log-junit junit.xml

</issue_description>

Comments on the Issue (you are @claude[agent] in this section)

@Claude Claude AI linked an issue May 25, 2026 that may be closed by this pull request
Claude AI and others added 2 commits May 25, 2026 08:14
- Changed input from coverage-xml-dir to coverage-file
- Updated PrTestCoverageAction to use CloverParser instead of PhpUnitXmlParser
- Updated action.yml, types.ts, index.ts to use new input name
- Updated PR check workflow to use clover.xml file
- Updated clover.xml test data to include all 3 files (service2.php was missing)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Agent-Logs-Url: https://github.com/shoptet/pr-test-coverage/sessions/11977e99-e3e3-4bed-8131-321bc9d59b8a

Co-authored-by: simonjur <118170619+simonjur@users.noreply.github.com>
- Updated all references from PHPUnit XML directory to Clover XML file
- Changed input parameter from coverage-xml-dir to coverage-file
- Updated usage examples for PHP, Node.js, and Python
- Clarified supported Clover XML metrics
- Updated failure conditions and descriptions

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Agent-Logs-Url: https://github.com/shoptet/pr-test-coverage/sessions/11977e99-e3e3-4bed-8131-321bc9d59b8a

Co-authored-by: simonjur <118170619+simonjur@users.noreply.github.com>
@Claude Claude AI requested a review from simonjur May 25, 2026 08: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.

switch xml format to junit

2 participants