Skip to content

Move back doctest instructions to setup.cfg#22587

Merged
sgugger merged 1 commit into
mainfrom
setup_cfg
Apr 5, 2023
Merged

Move back doctest instructions to setup.cfg#22587
sgugger merged 1 commit into
mainfrom
setup_cfg

Conversation

@sgugger

@sgugger sgugger commented Apr 5, 2023

Copy link
Copy Markdown
Collaborator

What does this PR do?

As a result of #22539, the options we have for the doctests are now all ignored. This PR reverts the change for those and puts them back in setup.cfg.

@sgugger sgugger requested review from amyeroberts and ydshieh April 5, 2023 11:46

@ydshieh ydshieh 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.

Confirmed it works (at least for 3 failing tests showing up after #22539). Thanks!

@sgugger sgugger merged commit 6c640f0 into main Apr 5, 2023
@sgugger sgugger deleted the setup_cfg branch April 5, 2023 11:53
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

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