Skip to content

Allow more artifacts to be download in CI#45629

Merged
ydshieh merged 1 commit intomainfrom
allow_more_artifiacts
Apr 24, 2026
Merged

Allow more artifacts to be download in CI#45629
ydshieh merged 1 commit intomainfrom
allow_more_artifiacts

Conversation

@ydshieh
Copy link
Copy Markdown
Collaborator

@ydshieh ydshieh commented Apr 24, 2026

What does this PR do?

We have more than 1000 artifacts in a daily CI run ...

Without this change, some artifacts won't be downloaded, then either the report statistic is wrong or the check new failing tests job fails.

We might change all v4 to v8 to be consistent - but I will wait until @paulinebm back.

@ydshieh ydshieh requested a review from tarekziade April 24, 2026 12:20
Comment on lines +136 to +137
env:
ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT: 2000
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT only works with v8

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

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
Copy Markdown
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.

LGTM thanks!

@ydshieh ydshieh merged commit 23ca437 into main Apr 24, 2026
18 checks passed
@ydshieh ydshieh deleted the allow_more_artifiacts branch April 24, 2026 14:03
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