Skip to content

Conversation

sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Sep 19, 2025

Description

With all eligible test suites now moved to toxgen, we can remove our old way of testing for regressions in new versions of packages.

For context, before toxgen we had two groups of test targets for each test suite: pinned and latest. Pinned targets used to be a small selection of hand-picked pinned versions of the package to test. Latest always pulled the newest release, to check for incompatibilities. There was a lot of code to cater to having these two different categories.

With the switch to toxgen, all targets are now pinned, and we make sure we pick a representative sample of versions to test against, always including the latest available version to detect regressions. I'm removing the code dealing with the latest group entirely, and removing references to anything pinned as this is now simply the default.

Issues

Reminders

@sentrivana sentrivana changed the title tests: Remove old system for checking for latest regressions tests: Remove old system for checking for new regressions Sep 19, 2025
@sentrivana sentrivana changed the title tests: Remove old system for checking for new regressions tests: Remove old system for checking for regressions Sep 19, 2025
@sentrivana sentrivana marked this pull request as ready for review September 19, 2025 13:07
@sentrivana sentrivana requested a review from a team as a code owner September 19, 2025 13:07
@sl0thentr0py
Copy link
Member

think someone with gh admin will need to update the checks?

@sentrivana
Copy link
Contributor Author

think someone with gh admin will need to update the checks?

Yeah will bother Johannes with this on Mon

@sentrivana sentrivana merged commit a926640 into master Sep 22, 2025
114 checks passed
@sentrivana sentrivana deleted the ivana/toxgen/cleanup-old-stuff branch September 22, 2025 10:13
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.

3 participants