Skip to content
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

Fix Error not captured in PR doctesting #26215

Merged
merged 4 commits into from
Sep 19, 2023
Merged

Fix Error not captured in PR doctesting #26215

merged 4 commits into from
Sep 19, 2023

Conversation

ydshieh
Copy link
Collaborator

@ydshieh ydshieh commented Sep 18, 2023

What does this PR do?

In https://app.circleci.com/pipelines/github/huggingface/transformers/72920/workflows/33795bcf-6b08-4c2f-b040-aa54bb44b12f/jobs/921247/artifacts,

we have errors.txt but not failures_short.txt. But we only checked the file failures_short.txt, therefore the error is not detected, and showing all test passed.

This PR fixes this.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 18, 2023

The documentation is not available anymore as the PR was closed or merged.

@ydshieh ydshieh force-pushed the fix_error_not_captured branch from ce6c54d to 0eae559 Compare September 18, 2023 12:14
@ydshieh
Copy link
Collaborator Author

ydshieh commented Sep 19, 2023

Error should have been reported in the console of https://app.circleci.com/pipelines/github/huggingface/transformers/72920/workflows/33795bcf-6b08-4c2f-b040-aa54bb44b12f/jobs/921247/steps, but it is not.

(The artifact has errors.txt file)

@ydshieh ydshieh requested a review from LysandreJik September 19, 2023 05:38
Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

GitHub makes it look like quite a hefty change even though it's mostly reordering. LGTM!

@ydshieh ydshieh merged commit 39df4ec into main Sep 19, 2023
MKhalusova pushed a commit to MKhalusova/transformers that referenced this pull request Sep 19, 2023
* fix

* fix

* fix

* fix

---------

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
parambharat pushed a commit to parambharat/transformers that referenced this pull request Sep 26, 2023
* fix

* fix

* fix

* fix

---------

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
blbadger pushed a commit to blbadger/transformers that referenced this pull request Nov 8, 2023
* fix

* fix

* fix

* fix

---------

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
EduardoPach pushed a commit to EduardoPach/transformers that referenced this pull request Nov 18, 2023
* fix

* fix

* fix

* fix

---------

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
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