Closed mborsetti closed 8 months ago
This is wonderful.
Unfortunately, I've noticed this pull request after https://github.com/Alir3z4/html2text/pull/358#issuecomment-1868045215 (I reverted https://github.com/Alir3z4/html2text/pull/373 along the way to make the CI pass, please feel free to return it in this pull request if it won't cause CI failure).
Could you please update the pull request?
This is wonderful.
Unfortunately, I've noticed this pull request after #358 (comment) (I reverted #373 along the way to make the CI pass, please feel free to return it in this pull request if it won't cause CI failure).
Could you please update the pull request?
What is the timeframe for a new release?
I don't see this project as being actively maintained (its last release is practically 4 years old)
I will be making a new release right after merging this pull request.
I will be making a new release right after merging this pull request.
Sounds good.
I've updated the PR and it now supports Python 3.11 and 3.12 too.
CI (tox) runs on local environment.
Thank you for this update. Coincidentally, this is happening at the right time for us so we can update our dependencies :-) Any plans merging this and creating the new release? Is there something can be done to help speeding this up?
.travis.yml
,setup.cfg
andtox.ini
files)In the process of implementing the above, this fix was required:
br_inside_a
(my creation, not sure how this wrong version got pushed and passed Travis CI)tox.ini
as Unicode tests would fail due to encoding defaulting to windows-1252 on Windows.gitattribute
to have correct line endingsIn the process of fixing the above, the following was discovered:
assert result == actual
but pytest standard -- and IDESs -- expect the reverse order, i.e.assert actual == expected
)In the process of all the above the following was in the way:
*.md
files to end with multiple empty lines and will automatically trim them out, so I had to modify tests toassert actual.rstrip() == expected.rstrip()
All
tox
tests are passed by this PR.