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

Add preview link to readthedocs build to PR description #378

Merged

Conversation

cary-ilm
Copy link
Member

Readthedocs triggers a build on PR, and a link is provided in the PR checks, but it's obscure. This action adds a link to the PR description, but only for PR's that actually modify the website.

Readthedocs triggers a build on PR, and a link is provided in the PR
checks, but it's obscure. This action adds a link to the PR
description, but only for PR's that actually modify the website.

Signed-off-by: Cary Phillips <[email protected]>
@cary-ilm cary-ilm merged commit 37ff823 into AcademySoftwareFoundation:main Mar 14, 2024
3 checks passed
cary-ilm added a commit that referenced this pull request Sep 4, 2024
* Add preview link to readthedocs build to PR description

Readthedocs triggers a build on PR, and a link is provided in the PR
checks, but it's obscure. This action adds a link to the PR
description, but only for PR's that actually modify the website.

Signed-off-by: Cary Phillips <[email protected]>

* fix typo: imath instead of openexr

Signed-off-by: Cary Phillips <[email protected]>

---------

Signed-off-by: Cary Phillips <[email protected]>
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.

2 participants