-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
TST: Decouple alldeps from oldestdeps #16950
Copy link
Copy link
Closed
Labels
Description
Problem: alldeps pulls in too many "optional" packages and there could be pinning conflicts with oldestdeps, causing unwanted version bumps. Example at #16903 (comment)
This combo was introduced in #12522 (comment) . It was a while ago when we did not have so many "optional" dependencies and that PR was to simplify jobs, so this combo was not necessarily to fix some specific problem we had per se.
Proposal: Remove alldeps from oldestdeps. Any "optional" dependencies we want to test there should be explicitly pinned in tox.ini instead. There are already 3 other jobs in regular CI alone that pulls in alldeps anyway, so I don't see the need to test alldeps against our oldestdeps.
Reactions are currently unavailable