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.

Sign up for GitHub

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

Jump to bottom

Support of JUnit.xml #35

Closed
wants to merge 17 commits into from
Closed

Support of JUnit.xml #35

wants to merge 17 commits into from

Conversation

jvdp1
Copy link
Member

@jvdp1 jvdp1 commented Aug 13, 2024

This PR tries to propose some improvements based on #27 led by @QGhd .

Changes from @QGhd 's PR:

  • IO limited as much as possible
  • New procedures to facilitate user experience

TODO

  • Addition of docstrings

@awvwgk @QGhd

ttv001 and others added 16 commits July 31, 2023 13:20
@jvdp1 jvdp1 requested a review from awvwgk August 13, 2024 13:48
@awvwgk awvwgk mentioned this pull request Sep 7, 2024
Merged
3 tasks
awvwgk
awvwgk requested changes Sep 7, 2024
View reviewed changes
Copy link
Member

@awvwgk awvwgk left a comment

Choose a reason for hiding this comment

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

This patch introduces a similar issue to test-drive as #27 since it can produce incomplete xml files if a test fails without properly returning an error. I hope I could reimplement the behavior from here in #42.

@awvwgk awvwgk closed this in #42 Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@awvwgk awvwgk awvwgk requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jvdp1 @awvwgk @QGhd