search

noseglid / atom-build

:hammer: Build your project directly from the Atom editor
https://atom.io/packages/build
MIT License
248 stars 97 forks source link

html error text #428

Closed oli-obk closed 8 years ago

oli-obk commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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

noseglid commented 8 years ago

Did you have any time to look into this?

oli-obk commented 8 years ago

not yet, I forgot about this...

alygin commented 8 years ago

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 commented 8 years ago

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