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

Already on GitHub? Sign in to your account

Send summary of test failures in notification email #30

Open
wants to merge 8 commits into
from

Conversation

Projects
None yet
1 participant

When we get the notification email for a failed build, we find it useful to be able to quickly see which tests failed, and why. So, I've added to the emails a summary of each failed test (description and failing reason). Maybe some people wouldn't find this useful, so I'd be happy to make this a user preference if you prefer - if not, then I'll remove the stubbed-out bit of code.

Would you be interested in pulling this? If so, I'll have to first check with my employer whether it's OK for me to sign the CLA.

I've finally been able to sign the CLA. There was some discussion with @puls going on about this request here (I accidentally submitted a request to my own repository and we ended up having the discussion there). I had a few small questions, which didn't get answered - if someone is able to answer those for me, I'll make the fixes and hopefully we can get this merged :-)

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