[tests] tighten dependency testing. by sayakpaul · Pull Request #13332 · huggingface/diffusers · GitHub
Skip to content

[tests] tighten dependency testing. - #13332

Merged
sayakpaul merged 7 commits into
mainfrom
revisit-deps-tests
Apr 10, 2026
Merged

sayakpaul merged 7 commits into
mainfrom
revisit-deps-tests

Conversation

@sayakpaul

@sayakpaul sayakpaul commented Mar 25, 2026 •

Copy link
Copy Markdown
Member

What does this PR do?

We should be able to catch stuff like #13321.

Additionally, I ported the existing dependency test suite to use pytest.

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

- main
paths:
- "src/diffusers/**.py"
- "tests/**.py"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I initially did it to trigger the tests in this PR. Will remove before merging.

@sayakpaul
sayakpaul requested a review from DN6 March 25, 2026 08:54

@DN6 DN6 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks 👍🏽

@github-actions github-actions Bot added tests pipelines CI size/M PR with diff < 200 LOC labels Apr 10, 2026
@sayakpaul

Copy link
Copy Markdown
Member Author

@sayakpaul
sayakpaul merged commit 896fec3 into main Apr 10, 2026
15 of 19 checks passed
@sayakpaul
sayakpaul deleted the revisit-deps-tests branch April 10, 2026 14:31
terarachang pushed a commit to terarachang/diffusers that referenced this pull request Apr 30, 2026
* tighten dependency testing.

* invoke dependency testing temporarily.

* f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI pipelines size/M PR with diff < 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants