Skip to content

Escape HTML in HTML reports #139

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

Closed

Conversation

reavowed
Copy link

@reavowed reavowed commented Oct 5, 2015

Fixes #138. Pretty easy 1 line change :) I refactored the test class a bit to allow a second test case - apologies for that being the bulk of this commit!

@reavowed reavowed force-pushed the escape-html-in-html-report branch from 670e582 to c6d6341 Compare October 6, 2015 09:41
@reavowed reavowed changed the title #138 Escape HTML in HTML reports Fix #138 - escape HTML in HTML reports Oct 6, 2015
@reavowed reavowed changed the title Fix #138 - escape HTML in HTML reports Escape HTML in HTML reports Oct 6, 2015
@sksamuel
Copy link
Member

I'll merge this if the conflicts are fixed.

@rorygraves
Copy link
Contributor

Ok, will sort the rebase of this.

@hiraiva
Copy link

hiraiva commented Aug 6, 2016

I've rebased this in #171

@gslowikowski
Copy link
Member

Fixed by #171

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

Successfully merging this pull request may close these issues.

5 participants