X Tutup
Skip to content
/ lxml Public

Avoid usage of unittest.makeSuite, removed from Python 3.13#385

Merged
scoder merged 1 commit intolxml:masterfrom
hroncok:python3.13
Oct 30, 2023
Merged

Avoid usage of unittest.makeSuite, removed from Python 3.13#385
scoder merged 1 commit intolxml:masterfrom
hroncok:python3.13

Conversation

@hroncok
Copy link
Contributor

@hroncok hroncok commented Oct 30, 2023

Patch generated by:

sed -i 's/unittest.makeSuite/unittest.defaultTestLoader.loadTestsFromTestCase/g' $(grep -rl makeSuite)

No idea if this will work on Python 2, hence only opening as a draft.

Patch generated by:

    sed -i 's/unittest.makeSuite/unittest.defaultTestLoader.loadTestsFromTestCase/g' $(grep -rl makeSuite)
@scoder
Copy link
Member

scoder commented Oct 30, 2023

Thanks. I've already started working on the port, while testing Cython in 3.13. The 3.11 test failures are known.

I might consider finally dropping Py2.7 support in the upcoming lxml 5.0. Seems a good time to do it.

@hroncok
Copy link
Contributor Author

hroncok commented Oct 30, 2023

Anyway, this seems to actually work with 2.7.

@hroncok hroncok marked this pull request as ready for review October 30, 2023 17:13
@scoder scoder merged commit 3418796 into lxml:master Oct 30, 2023
@scoder
Copy link
Member

scoder commented Oct 30, 2023

Thanks

@hroncok hroncok deleted the python3.13 branch October 31, 2023 23:01
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

X Tutup