Mark<ol reversed> as a boolean attribute so it serializes properly in HTML.
Allow<ol reversed> in the sanitizer.ClosesSanitizer removes the <ol> "reversed" attribute #321.
Allow<ol start> in the sanitizer.
<ol type> was already allowed, but probably accidentally (type is an attribute allowed for other tags). I added a test to prevent it from regressing in case we add per-element attribute sanitization in the future.

https://html.spec.whatwg.org/multipage/grouping-content.html#attr-ol-reversed

Add support for <ol reversed>, related attributes

40bd63e

* Mark <ol reversed> as a boolean attribute so it serializes properly in  HTML.* Allow <ol reversed> in the sanitizer.Closeshtml5lib#321.* Allow <ol start> in the sanitizer.* <ol type> was already allowed, but probably accidentally (type is an  attribute allowed for other tags). I added a test to prevent it from  regressing in case we add per-element attribute sanitization in the  future.https://html.spec.whatwg.org/multipage/grouping-content.html#attr-ol-reversed

twm added a commit to twm/html5lib-python that referenced this pull request

Jun 10, 2018

Remove Appveyor Python 3.3 builds

e137495

It looks like this was missed inhtml5lib#358. It is causing build failures inhtml5lib#395 andhtml5lib#396.

twm mentioned this pull request

Jun 10, 2018

Remove Appveyor Python 3.3 builds#397

Merged

gsnedders pushed a commit that referenced this pull request

Oct 1, 2018

Remove Appveyor Python 3.3 builds (#397)

7facf98

It looks like this was missed in#358. It is causing build failures in#395 and#396.

Merge branch 'master' into ol-reversed

4b34642

Copy link

codecov-io commentedOct 7, 2018•
edited
Loading

Codecov Report

Merging#396 intomaster willincrease coverage by0.01%.
The diff coverage is100%.

@@            Coverage Diff            @@##           master    #396      +/-   ##=========================================+ Coverage   90.79%   90.8%   +0.01%=========================================  Files          50      50                Lines        6952    6964      +12       Branches     1328    1328              =========================================+ Hits         6312    6324      +12  Misses        481     481                Partials      159     159

Impacted Files	Coverage Δ
html5lib/filters/sanitizer.py	`93.16% <ø> (ø)`	⬆️
html5lib/constants.py	`100% <ø> (ø)`	⬆️
html5lib/tests/test_sanitizer.py	`95.34% <100%> (+0.75%)`	⬆️

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update4f92357...4b34642. Read thecomment docs.