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

Use HTML5 Parser (fixes #889) #896

Merged
merged 8 commits into from Aug 22, 2017
Merged

Use HTML5 Parser (fixes #889) #896

merged 8 commits into from Aug 22, 2017

Conversation

dac514
Copy link
Contributor

@dac514 dac514 commented Aug 22, 2017

Update DomDocument's loadHTML function with Mastermind's HTML5 parser where appropriate (fix #889).

@dac514 dac514 changed the title Use HTML5 Parser Use HTML5 Parser (fixes #889) Aug 22, 2017
@codecov
Copy link

codecov bot commented Aug 22, 2017

Codecov Report

Merging #896 into dev will increase coverage by 0.27%.
The diff coverage is 35.59%.

@@             Coverage Diff              @@
##                dev     #896      +/-   ##
============================================
+ Coverage     30.74%   31.02%   +0.27%     
- Complexity     2877     2879       +2     
============================================
  Files            82       82              
  Lines         17832    17822      -10     
============================================
+ Hits           5483     5529      +46     
+ Misses        12349    12293      -56

@greatislander greatislander merged commit 8596c2f into dev Aug 22, 2017
@greatislander greatislander deleted the HTML5-parser branch August 22, 2017 16:41
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 this pull request may close these issues.

Use Masterminds\HTML5 Instead Of DOMDocument With Hacks
2 participants