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

Clarification on PR build webhooks and GitHub Action #11998

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

k4kfh
Copy link

@k4kfh k4kfh commented Feb 15, 2025

I was setting up pull request builds and the Pull Request Build Preview GitHub Action and ran into some trouble. It ended up being that my repository's webhook was set to only send push events, not all events, so I guess RTD was not getting notified of pull requests and therefore not triggering PR builds (despite having PR builds turned on in the RTD project settings).

This is a quick shot at clarifying this behavior on the docs page. I also added some clarification on the GitHub Action, because I feel it's a fantastic feature, but it's a little hard to find if you don't already know what you're looking for.

Let me know what you think - just wanted to get this going while it was fresh on my mind.


📚 Documentation previews 📚

Adding additional detail to aid in troubleshooting PR builds, and the PR build preview GitHub Action.
@k4kfh k4kfh requested a review from a team as a code owner February 15, 2025 00:00
@k4kfh k4kfh requested a review from ericholscher February 15, 2025 00:00
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