Skip to content

Conversation

@rffontenelle
Copy link
Collaborator

@rffontenelle rffontenelle commented Nov 28, 2024

When on pull request, this builds the webpage in readthedocs and adds preview link.

This was suggested by @hugovk in https://github.com/m-aciek/pydocs-translation-dashboard/pull/11#issuecomment-2502998463 and inspired in CPython's solution.

Example of build with everything properly set.

@m-aciek m-aciek merged commit 5d16e73 into main Nov 29, 2024
1 check passed
@m-aciek m-aciek deleted the add-readthedocs-preview branch November 29, 2024 09:36
@m-aciek
Copy link
Collaborator

m-aciek commented Nov 29, 2024

Thanks!

@m-aciek
Copy link
Collaborator

m-aciek commented Dec 6, 2024

@rffontenelle could you please add me to the Read the Docs project? I am maciek on readthedocs.org.

@rffontenelle
Copy link
Collaborator Author

rffontenelle commented Dec 6, 2024

Done. Invited to https://app.readthedocs.org/projects/pydocs-translation-dashboard/

@m-aciek
Copy link
Collaborator

m-aciek commented Dec 9, 2024

Thanks. The preview doesn't work for https://github.com/m-aciek/pydocs-translation-dashboard/pull/15. Is there some config step required?

@m-aciek
Copy link
Collaborator

m-aciek commented Dec 9, 2024

Ok, I think we (you?) found the cause, the webhook integration. Let's wait for the next PR.

@m-aciek
Copy link
Collaborator

m-aciek commented Dec 13, 2024

Out of curiosity -- what was missing here?

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.

4 participants