omit unnecessary spaces around <b>, <em>, and <strike>

Alir3z4 / html2text

Convert HTML to Markdown-formatted text.

alir3z4.github.io/html2text/

GNU General Public License v3.0

1.76k stars 270 forks source link

omit unnecessary spaces around <b>, <em>, and <strike> #342

Closed snarfed closed 3 years ago

snarfed commented 3 years ago

fixes #324

coveralls commented 3 years ago

Coverage increased (+0.2%) to 97.026% when pulling d127b7720d14e4ff53ccfc426679ac28bfba73e6 on snarfed:no-inline-space into 4592133f82dfea65491321c28dc0f080fc786f49 on Alir3z4:master.

coveralls commented 3 years ago

Coverage decreased (-96.8%) to 0.0% when pulling 8e8c1bc250f4636092ce0714b9ad425deabed38f on snarfed:no-inline-space into 4235c1a31b18ef3d3d81434626e8d3778710866d on Alir3z4:master.

coveralls commented 3 years ago

Coverage decreased (-96.8%) to 0.0% when pulling 8e8c1bc250f4636092ce0714b9ad425deabed38f on snarfed:no-inline-space into 4235c1a31b18ef3d3d81434626e8d3778710866d on Alir3z4:master.

djdembeck commented 3 years ago

Noticed these problems as well. Any chance this can be merged?

Alir3z4 commented 3 years ago

@snarfed @djdembeck just merged this in and I noticed it failed on master. Could you please let me know if you could troubleshoot?

Alir3z4 commented 3 years ago

The failure can be seen on travis build report https://travis-ci.org/github/Alir3z4/html2text/builds/769590651?utm_source=github_status&utm_medium=notification

snarfed commented 3 years ago

Sure! Done, in #357. Looks like this one collided with #333, which was merged after this PR passed CI but before it was merged.

Alir3z4 commented 3 years ago

Excellent!