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

Invalid rowspans #183

Merged
merged 2 commits into from Nov 26, 2018

Conversation

@j-rausch
Copy link
Contributor

commented Nov 26, 2018

Some html documents with invalid rowspans/colspans (e.g. rowspan="2]", rowspan="") would crash the parser. I added some simple error handling to utiilize the default spans in such cases.

@lukehsiao lukehsiao added the bug label Nov 26, 2018
@lukehsiao lukehsiao added this to the v0.3.7 milestone Nov 26, 2018
@lukehsiao lukehsiao merged commit 72c0ed6 into HazyResearch:master Nov 26, 2018
1 of 2 checks passed
1 of 2 checks passed
coverage/coveralls Coverage decreased (-0.06%) to 78.827%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.