search

rbanffy / pip-chill

A more relaxed `pip freeze`
Other
555 stars 25 forks source link

Add setuptools req and tox py312 #75

Closed MarcinKonowalczyk closed 2 weeks ago

MarcinKonowalczyk commented 2 months ago

Resolves #74 in the easy way (add setuptools dep for py>3.12).

Also encountered a failing test with --show-all option in test_pip_chill::TestPip_chill::test_all (line 34). It requires both pip-chill and pip to appear in the output, but that's not true if pip is a dependency (e.g. of package pip-autoremove). I've added a bit to elevate pip (as well as the other "special" packages) to distributions list (also explained in a comment).

MarcinKonowalczyk commented 2 months ago
Screenshot 2024-04-28 at 17 59 24
MarcinKonowalczyk commented 2 months ago

also updated the tests to the more modern pytest style. tox still runs fine on all versions and the actual content of the tests is unchanged. it just makes the diff a bit larger. its all in one commit and v happy for that to be reverted tho.

MarcinKonowalczyk commented 3 weeks ago

@rbanffy review pls? :))

MarcinKonowalczyk commented 2 weeks ago

tox still all green ✅

MarcinKonowalczyk commented 2 weeks ago

Made an issue (#78) describing/reproducing the case for special packages elevation.