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

Add a job to test the documentation build #14645

Merged
merged 3 commits into from
Dec 6, 2021
Merged

Add a job to test the documentation build #14645

merged 3 commits into from
Dec 6, 2021

Conversation

sgugger
Copy link
Collaborator

@sgugger sgugger commented Dec 6, 2021

What does this PR do?

This PR adds a GitHub action to test the documentation build. While we will in the future have something to check the built documentation, it's important to test right now that there is no failure in the build at least, even if we can't see the result.

@sgugger sgugger requested a review from LysandreJik December 6, 2021 17:56
@sgugger
Copy link
Collaborator Author

sgugger commented Dec 6, 2021

The caching only gets us one minute back in the setup :-(
Once the dummies have the docstrings of their real counterpart, we can stop installing all dependencies and have better timing though!

@sgugger sgugger merged commit 03fda7b into master Dec 6, 2021
@sgugger sgugger deleted the test_doc_build branch December 6, 2021 18:56
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.

1 participant