-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add details to error message for junit #7390
Conversation
Fixes issue #7385
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 @dajose, looks good to me!
Could you also add yourself to AUTHORS
?
(We should squash-merge this) |
thanks @nicoddemus :) :) I'm already on the authors 🤓 |
Oh sorry, great then! 😁 |
@nicoddemus excuse me being intense... but are we waiting for more reviewers? or maybe something on my side? |
Not at all, don't worry about pinging!
The former. We usually wait a bit to see if other maintainers want to take a second look. I will merge this tomorrow unless someone else wants more time to review this. 👍 |
Thanks @dajose! |
thanks to you @nicoddemus :) |
Co-authored-by: Bruno Oliveira <nicoddemus@gmail.com>
closes #7385