-
Notifications
You must be signed in to change notification settings - Fork 322
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
Enable the ReadTheDocs PR Preview #1580
Labels
Comments
I enabled the setting! |
Thanks! Looks like the check is not visible though, for example on the recent #1584: Not sure there is something else to enable to make it visible. |
Yeah, I just tried refreshing the web hook from the github side so
hopefully it will work now?
…On Wed, May 18, 2022, 9:04 AM Jeremy Tuloup ***@***.***> wrote:
Thanks!
Looks like the check is not visible though, for example on the recent
#1584 <#1584>:
[image: image]
<https://user-images.githubusercontent.com/591645/168989079-1f0fdd90-8731-44d0-a6df-82335d639814.png>
Not sure there is something else to enable to make it visible.
—
Reply to this email directly, view it on GitHub
<#1580 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAUL5CIGBPJ3KUE4Z55DKLVKSQB3ANCNFSM5VU7O6SA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Yes, looking good on #1586 now: Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The ReadTheDocs PR Preview feature is really useful to automatically build the docs when a new PR is opened.
We should consider enabling it for this repo. This would make reviewing documentation changes easier without having to build the changes locally. For example for #1577 which updated the way the changelog is handled.
Documentation on RTD: https://docs.readthedocs.io/en/stable/pull-requests.html
The text was updated successfully, but these errors were encountered: