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.

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

html error text #428

Closed
oli-obk opened this issue May 20, 2016 · 6 comments · Fixed by #455
Closed

html error text #428

oli-obk opened this issue May 20, 2016 · 6 comments · Fixed by #455

Comments

@oli-obk
Copy link
Contributor

oli-obk commented May 20, 2016

Since linter supports html error text, would it make sense to add a html_message field/regex match ?

Note: my actual use case is markdown, but converting that to html is trivial.

@noseglid
Copy link
Owner

You should be able to pass HTML as it is if you're using the build files programmatically (or via a build provider). Does this not suffice?

@oli-obk
Copy link
Contributor Author

oli-obk commented May 22, 2016

I don't think it works, linter has an extra field for html. But I'll check

@noseglid
Copy link
Owner

Did you have any time to look into this?

@oli-obk
Copy link
Contributor Author

oli-obk commented Jun 28, 2016

not yet, I forgot about this...

@alygin
Copy link
Contributor

alygin commented Jul 8, 2016

Linter escapes html before showing the message. It would be really handy to be able to pass an html to Linter's html field.

@alygin
Copy link
Contributor

alygin commented Jul 8, 2016

Here's the PR in case you decide to add such support. I've checked, it works correctly under various circumstances.

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 a pull request may close this issue.

3 participants