Skip to content

remove dead deepfloyd if shared test mixin - #14611

Merged
sayakpaul merged 1 commit into
huggingface:mainfrom
akshan-main:remove-if-shared-test-mixin
Aug 27, 2026
Merged

remove dead deepfloyd if shared test mixin#14611
sayakpaul merged 1 commit into
huggingface:mainfrom
akshan-main:remove-if-shared-test-mixin

Conversation

@akshan-main

Copy link
Copy Markdown
Contributor

What does this PR do?

Removes the IFPipelineTesterMixin from the deepfloyd_if test package __init__.py. All six IF test files migrated to the new mixin structure with their components inlined (#14580, #14596, #14602, #14603, #14604, #14606), so nothing imports it anymore. Follows #14606.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline?
  • Did you read our philosophy doc (important for complex PRs)?
  • Was this discussed/approved via a GitHub issue or the forum? Discussed on Slack with @sayakpaul
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

Who can review?

@sayakpaul

@github-actions github-actions Bot added tests size/L PR with diff > 200 LOC labels Aug 26, 2026

@sayakpaul sayakpaul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superb!

@sayakpaul
sayakpaul merged commit ec94eec into huggingface:main Aug 27, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants