Skip to content

Commit 80367b4

Browse files
committed
.
1 parent eef0511 commit 80367b4

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

scripts/populate_tox/populate_tox.py

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -226,11 +226,15 @@ def _supports_lowest(release: Version) -> bool:
226226
return releases, latest_prerelease
227227

228228

229-
def pick_releases_to_test(releases: list[Version]) -> list[Version]:
229+
def pick_releases_to_test(
230+
releases: list[Version], last_prerelease: Optional[Version]
231+
) -> list[Version]:
230232
"""Pick a handful of releases to test from a sorted list of supported releases."""
231233
# If the package has majors (or major-like releases, even if they don't do
232234
# semver), we want to make sure we're testing them all. If not, we just pick
233235
# the oldest, the newest, and a couple in between.
236+
#
237+
# If there is a relevant prerelease, also test that in addition to the above.
234238
has_majors = len(set([v.major for v in releases])) > 1
235239
filtered_releases = set()
236240

@@ -265,7 +269,11 @@ def pick_releases_to_test(releases: list[Version]) -> list[Version]:
265269
releases[-1], # latest
266270
}
267271

268-
return sorted(filtered_releases)
272+
filtered_releases = sorted(filtered_releases)
273+
if last_prerelease is not None:
274+
filtered_releases.append(last_prerelease)
275+
276+
return filtered_releases
269277

270278

271279
def supported_python_versions(
@@ -517,9 +525,7 @@ def main() -> None:
517525
# Pick a handful of the supported releases to actually test against
518526
# and fetch the PyPI data for each to determine which Python versions
519527
# to test it on
520-
test_releases = pick_releases_to_test(releases)
521-
if latest_prerelease is not None:
522-
test_releases.append(latest_prerelease)
528+
test_releases = pick_releases_to_test(releases, latest_prerelease)
523529

524530
for release in test_releases:
525531
_add_python_versions_to_release(integration, package, release)

0 commit comments

Comments
 (0)