search

validator / htmlparser

The Validator.nu HTML parser https://about.validator.nu/htmlparser/
Other
56 stars 26 forks source link

Conform ambiguous-ampersand reporting to HTML spec #27

Closed sideshowbarker closed 4 years ago

sideshowbarker commented 4 years ago

This is a replacement change for the previous ambiguous-ampersand attempt in the validator-nu branch. Unlike that previous attempt, this one, I think, will not have the problems the Mozilla try bot uncovered.

https://treeherder.mozilla.org/#/jobs?repo=try&selectedTaskRun=GdCS5FrVRbCXm2jscZdZkw.0&revision=59a1564

hsivonen commented 4 years ago

New try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=439fd78375fe4d83a5999e2e0c9d30075da4efb3

sideshowbarker commented 4 years ago

New try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=439fd78375fe4d83a5999e2e0c9d30075da4efb3

Thanks — watching it now

sideshowbarker commented 4 years ago

Just FYI for now, from https://treeherder.mozilla.org/#/jobs?repo=try&revision=439fd78375fe4d83a5999e2e0c9d30075da4efb3&selectedTaskRun=dKhZOkS3SAqKuh4HMwuHfw.0 I realize this is still failing. So I’m right now looking back at the code (and the spec) and will make another attempt to get it working the way it should be.

sideshowbarker commented 4 years ago

Closing in favor of #30 (which I think fixes all the test failures)