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.

Sign up for GitHub

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

Jump to bottom

Implement ParserStream for HTML fragments #227

Closed
ComFreek opened this issue Dec 26, 2017 · 0 comments · Fixed by #487
Closed

Implement ParserStream for HTML fragments #227

ComFreek opened this issue Dec 26, 2017 · 0 comments · Fixed by #487
Labels
enhancement

Comments

@ComFreek
Copy link

ComFreek commented Dec 26, 2017 •
edited
Loading

ParserStream (docs, code) does not currently support parsing HTML fragments.
At least that's the impression I got looking at the docs - which don't mention it - and at the code, which is not nearly as complex as parseFragment.

The use case would be to allow for a streaming load method in Cheerio: cheeriojs/cheerio#1126 (comment)

What do you think?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(parser-stream): Support parsing fragments inikulin/parse5
2 participants
@RReverser @ComFreek