-
Notifications
You must be signed in to change notification settings - Fork 5
Add the readthedocs preview back #154
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
base: main
Are you sure you want to change the base?
Conversation
| documentation-links: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: readthedocs/actions/preview@v1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This action is deprecated.
We could replace this workflow with the new app?
See python/core-workflow#587 for details.
|
Co-authored-by: Stan Ulbrych <[email protected]>
This reverts commit 325faab.
|
Ping @AA-Turner for the above. |
This reverts commit 442383e The second script is too slow for our RTD tier
|
@AA-Turner could you please install Read the Docs Community GitHub application for the python-docs-translations org at a spare moment? The build details generation script makes RTD build to timeout, I removed its invocation in the config. When removed, 404s will be shown for build details, and changes in details view won't be seen in RTD summary. |
The GH pages preview doesn't work for forks. We could change it to render the previews on forks' GH pages, but that would require enabling actions and GH pages on forks which is an extra effort.
I fixed the RTD project configuration after the repository move (Repository URL config which wasn't available to change earlier?), let's bring back the RTD previews for a better developer experience.
📊 Dashboard preview 📊: https://python-docs-translations.github.io/dashboard/154/merge/