You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We recently had some issues/discussion around what versions of package dependencies we should be running tests against. There are at least 3 distinct cases:
(1) Running against the versions pinned in the lockfile, which should be the same across platforms,
(2) Running against the oldest versions of packages we say we support,
(3) Running against the newest versions of packages we say we support.
I believe we can configure the actions to do any of these, or some combination.
@fcollman at some point mentioned that the PR tests could be (1) while the daily builds or pre-release builds could use (3). I worry that this division could make errors slower to catch in the development cycle, since they'd only pop up after PRs are merged.
Soliciting thoughts in general!
The text was updated successfully, but these errors were encountered:
As an example of why (2) could be helpful, this bugfix was because I made some assumptions about urllib 2 that I didn't realize weren't part of urllib 1 since I never tested against it #239
We recently had some issues/discussion around what versions of package dependencies we should be running tests against. There are at least 3 distinct cases:
(1) Running against the versions pinned in the lockfile, which should be the same across platforms,
(2) Running against the oldest versions of packages we say we support,
(3) Running against the newest versions of packages we say we support.
I believe we can configure the actions to do any of these, or some combination.
@fcollman at some point mentioned that the PR tests could be (1) while the daily builds or pre-release builds could use (3). I worry that this division could make errors slower to catch in the development cycle, since they'd only pop up after PRs are merged.
Soliciting thoughts in general!
The text was updated successfully, but these errors were encountered: