Skip to content

regression-test :: move more methods to ExtractionTest #1161

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dev-mlb
Copy link
Collaborator

@dev-mlb dev-mlb commented Jul 10, 2025

Builds on #1153 and #1160 -- after this pr, regression test will only contain a few overridden methods.

@dev-mlb dev-mlb added the test-only The change only impacts test code label Jul 10, 2025
@jpdahlke jpdahlke added the rebase Things have happened and now a rebase is needed label Jul 11, 2025
fbruton
fbruton previously approved these changes Jul 11, 2025
@fbruton fbruton self-requested a review July 11, 2025 18:20
drivenflywheel
drivenflywheel previously approved these changes Jul 11, 2025
@dev-mlb dev-mlb dismissed stale reviews from drivenflywheel and fbruton July 11, 2025 19:55

The merge-base changed after approval.

@dev-mlb dev-mlb removed the rebase Things have happened and now a rebase is needed label Jul 11, 2025
@jpdahlke jpdahlke added this to the v8.32.0 milestone Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test-only The change only impacts test code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants