Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Glenn Washburn's JUnit patch was never merged back then. What happened? #334

Open
joakimr-axis opened this issue Jun 1, 2021 · 2 comments

Comments

@joakimr-axis
Copy link

For integration with various CI pipeline tools with JUnit format support, it would be nice to have JUnit output from check directly instead of relying on external converters such as check2junit.py. It seems from the mailinglist archives that Glenn Washburn did a patch for this back in the days

https://sourceforge.net/p/check/mailman/message/2963561/

and that it was positively received bu the maintainers. Apparently, a larger refactorization was done around that time; was that JUnit patch then never updated to match the new codebase and forgotten?

I did some experiments with applying it to the contemporary codebase, but there is a bit more than simple fixes needed to make things roll with the current codebase design. (And then some better knowledge of check and JUnit than mine is needed.)

@brarcher
Copy link
Contributor

brarcher commented Jun 2, 2021

was that JUnit patch then never updated to match the new codebase and forgotten?

My understanding from reading through the conversation is that the patches stalled out when asking for adding documentation to go along with adding support for JUnit XML output.

The Check project would be up for reviewing this feature again, if a volunteer from the community would be available to adapt the patches and create a pull request in GitHub.

@trevershick
Copy link

I didn't add the timing yet, but I have the Junit XML format being outputted in #337 . I'll add the timing after this initial PR, if it gets merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants