search

kangax / html-lint

Linter that was originally part of kangax/html-minifier
MIT License
47 stars 6 forks source link

Remove minifier code #1

Closed alexlamsl closed 8 years ago

alexlamsl commented 8 years ago

Now expose a new function lint(input, element) in addition to HTMLLint.

If element is omitted then return the lint errors as string (as opposed to console.log() previously).

The minor detail being that html-lint is already taken on NPM, so @kangax please come up with something 😅

alexlamsl commented 8 years ago

@kangax please turn on Travis CI for this repository 😉

kangax commented 8 years ago

My brain is fried, can you remind me how do I do that?

alexlamsl commented 8 years ago

@kangax sign in on https://travis-ci.org/kangax/html-lint and the green buttons should lead the way - I've got stuck because I'm not the owner of the project :sweat_smile:

alexlamsl commented 8 years ago

In fact it's https://travis-ci.org/profile after signing in

kangax commented 8 years ago

Enabled, so try pushing something and it should work

On Sat, Apr 16, 2016 at 2:17 AM, Alex Lam S.L. notifications@github.com wrote:

In fact it's https://travis-ci.org/profile after signing in

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/kangax/html-lint/pull/1#issuecomment-210745386

alexlamsl commented 8 years ago

@kangax works like a charm - thanks! :+1:

kangax commented 8 years ago

No, thanks to you for all the goodness :)

alexlamsl commented 8 years ago

CI passed - now we just need to name this project on NPM, as html-lint is already taken...

kangax commented 8 years ago

One dumb idea is perhaps html-minifier-lint?

On Sat, Apr 16, 2016 at 1:37 PM, Alex Lam S.L. notifications@github.com wrote:

CI passed - now we just need to name this project on NPM, as html-lint is already taken...

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/kangax/html-lint/pull/1#issuecomment-210862171

alexlamsl commented 8 years ago

Sounds logical enough :stuck_out_tongue_closed_eyes:

alexlamsl commented 8 years ago

@kangax published to npm, and added you as owner - I've put in as 0.9.0 for now, and I guess once we've done the corresponding removal work on html-minifer and verified that it all works we'll publish 1.0.0

kangax commented 8 years ago

Perfect, thanks!

Sent from my iPhone

On 16 Apr 2016, at 14:04, Alex Lam S.L. notifications@github.com wrote:

@kangax published to npm, and added you as owner - I've put in as 0.9.0 for now, and I guess once we've done the corresponding removal work on html-minifer and verified that it all works we'll publish 1.0.0

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub