Skip to content

Don't run tests_hub if no tests found#45014

Merged
ydshieh merged 3 commits intomainfrom
check_hub
Mar 26, 2026
Merged

Don't run tests_hub if no tests found#45014
ydshieh merged 3 commits intomainfrom
check_hub

Conversation

@ydshieh
Copy link
Collaborator

@ydshieh ydshieh commented Mar 26, 2026

What does this PR do?

#30674 refactors the way we obtain CircleCI test files to run for each job.

It always puts ["tests"] for tests_hub, so each commit of each PR will run it, no matter if there is any change to codebase.

Let's reduce the CI load by running an Empty job instead (just what it did before #30674), see

https://app.circleci.com/pipelines/github/huggingface/transformers/169582/workflows/506f301e-600c-4717-9c70-3edd9b48275c

@ydshieh ydshieh changed the title Check hub Don't run tests_hub if no tests found Mar 26, 2026
@ydshieh ydshieh requested a review from tarekziade March 26, 2026 08:55
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Collaborator

@tarekziade tarekziade left a comment

Choose a reason for hiding this comment

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

Thanks! just a small nit

@ydshieh ydshieh merged commit b1ba6d7 into main Mar 26, 2026
17 of 19 checks passed
@ydshieh ydshieh deleted the check_hub branch March 26, 2026 09:32
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.

4 participants