Skip to content

/ html5lib-python

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

Jump to bottom

Drop datrie support #455

Merged
merged 2 commits into from May 28, 2020
Merged

Drop datrie support #455

merged 2 commits into from May 28, 2020

Conversation

@gsnedders
Copy link
Member

@gsnedders gsnedders commented Feb 29, 2020

see #442
@gsnedders
Drop datrie support
Loading status checks…
d4a6617
@gsnedders gsnedders mentioned this pull request Feb 29, 2020
Closed
@gsnedders
fixup! Drop datrie support
Loading status checks…
0fe7acd
@codecov-commenter
Copy link

@codecov-commenter codecov-commenter commented May 24, 2020 •
edited

Codecov Report

Merging #455 into master will increase coverage by 0.30%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
+ Coverage   90.60%   90.90%   +0.30%     
==========================================
  Files          50       49       -1     
  Lines        6983     6930      -53     
  Branches     1328     1322       -6     
==========================================
- Hits         6327     6300      -27     
+ Misses        497      471      -26     
  Partials      159      159
Impacted Files Coverage Δ
html5lib/_trie/__init__.py 100.00% <100.00%> (ø)
html5lib/_inputstream.py 91.77% <0.00%> (-0.15%) ⬇️
html5lib/html5parser.py 98.44% <0.00%> (+1.19%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 05b73ef...0fe7acd. Read the comment docs.
@gsnedders gsnedders merged commit 5cd73ef into html5lib:master May 28, 2020
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@gsnedders
Copy link
Member Author

@gsnedders gsnedders commented May 28, 2020

Landed given no review forthcoming in three months.
@gsnedders gsnedders deleted the gsnedders:rm-datrie branch May 28, 2020
@gsnedders gsnedders mentioned this pull request Jun 10, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@gsnedders @codecov-commenter
You can’t perform that action at this time.