Added HTML parser support

[marcelaraujo]

:)

@stephanebachelier It's squashed into one single commit.

Thanks

[XhmikosR]

Why do we need run.js in root? Can't you move it in test/?

[marcelaraujo]

Sorry. I forgot remove it. It will be done.

[marcelaraujo]

It's done.

Will it be merged?

[XhmikosR]

I'll leave this to @stephanebachelier. But you should squash those 2 commits.

[marcelaraujo]

Squash again?

😓

Let's go...

[marcelaraujo]

It's done.

[marcelaraujo]

Any feedback guys?

I'm using my own repository but I wish move to the right target

😊

[stevemao]

Any reason you use parse5 instead of cheerio?

[marcelaraujo]

Only one reason, performance!

[stephanebachelier]

@marcelaraujo sorry for the delay, client work in the pipe. I really want to merge your work as it's a major feature and will solve lots of issues in this repo. I come back to you this week after having made some tests on real projects.

[stephanebachelier]

@marcelaraujo @XhmikosR I would like to have the #518 to pass before if it might be accepted, as I had hard time rebasing the work done in #306. This PR should be easy to rebase on #518.

[marcelaraujo]

Ok.

👍

[stephanebachelier]

@marcelaraujo BTW feel free to ping me if I ever get lost in client work :)

[marcelaraujo]

I don't think we will have problems between both PR's because we work on different files. Maybe some tests will break... maybe...

:)

[stephanebachelier]

@marcelaraujo I've created a dev branch to merge majors PR. Could you create another PR on top of dev branch. You can't edit a PR.