-
Notifications
You must be signed in to change notification settings - Fork 64
Add Support for xunit.v3 #339
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
Conversation
Add support for xunit.v3 Signed-off-by: Jonas Bengtsson <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution. Could you add a test project similar to the ones for MSTestV2 und NUnit?
Co-authored-by: Alexander Linne <[email protected]> Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #339 +/- ##
=======================================
Coverage 65.65% 65.65%
=======================================
Files 255 255
Lines 23032 23032
Branches 1962 1962
=======================================
Hits 15121 15121
Misses 7366 7366
Partials 545 545 ☔ View full report in Codecov by Sentry. |
@ljb Could you reformat the code once? Also we've removed the License Headers from the source files and the explicit authors from |
Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
@alexanderlinne: I have now removed the explicit authors and license code. I also reformated the code with Rider. That didn't change much code though. Since there is no |
You should be able to correctly reformet the code by running:
|
Signed-off-by: Jonas Bengtsson <[email protected]>
@alexanderlinne: I have now run Great stuff! 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick changes :) Just have to get the tests running and then we are good to go
Signed-off-by: Jonas Bengtsson <[email protected]> Co-authored-by: Alexander Linne <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]> Co-authored-by: Alexander Linne <[email protected]>
Add support for xunit.v3
Fixes: #338