-
-
Notifications
You must be signed in to change notification settings - Fork 165
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 pull request preview on Read the Docs #675
Conversation
Note: Requires enabling this feature on RTD by a Maintainer
@stevepiercy could you have a look at the failed RTD build? https://readthedocs.org/projects/icalendar/builds/24871814/ |
I got docs building on pull requests now. After this is merged, PRs should now get a comment added to the description as well. @niccokunzmann one more review, please, to ensure I caught everything. Also I don't think this needs to be documented any more than the comments in the file. |
Yes, this looks ok. |
It builds but I do not see a comment. I think, this is fine and if it needs any improvement, we can add this on the way... |
Note: Requires enabling this feature on RTD by a Maintainer.
Closes #669.