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

WIP: fix test suite #402

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

WIP: fix test suite #402

wants to merge 4 commits into from

Conversation

anarcat
Copy link

@anarcat anarcat commented Nov 2, 2018

Version 0.99999999 (eight nines) and up do not provide the html5lib.tokenizer API used by wpull.

This pull request fixes #332

I remembered to:

  • Update or add unit tests if needed (this should fix broken unit tests)
  • Update or add documentation/comments if needed (nothing needed)
  • Made sure stray files or whitespace didn't get committed (yes, yes)
  • If significant changes, branch from develop and set to merge into develop (nope)
  • Read the guidelines for contributing (yes, i did)

This was cherry-picked from #393 to at least get tests back online.

Version 0.99999999 (eight nines) and up do not provide the `html5lib.tokenizer` API used by wpull.
@anarcat anarcat changed the title Pin html5lib version to 0.9999999 (seven nines) (closes #332) WIP: fix test suite Nov 2, 2018
@anarcat
Copy link
Author

anarcat commented Nov 2, 2018

i'll see if i can pile a few commits to make travis green, ideas from @JustAnotherArchivist

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.

2 participants