search

holoviz-topics / examples

Visualization-focused examples of using HoloViz for specific topics
https://examples.holoviz.org
Creative Commons Attribution 4.0 International
82 stars 24 forks source link

infra: add comment on PRs to ask for dev site review #283

Closed maximlt closed 1 year ago

maximlt commented 1 year ago

I've observed it's too easy to forget reviewing the dev site when updating a project. This PR automatically adds a comment to the PR when the dev docs build is successful.

github-actions[bot] commented 1 year ago

Make sure to review the projects you changed on the dev site: https://holoviz-dev.github.io/examples/. You can also download an archive of the site from the workflow summary page, which comes in handy when your dev site built was overriden by another PR (we have a single dev site!).

github-actions[bot] commented 1 year ago

Your changes were successfully integrated in the dev site, make sure to review the pages of the projects you touched before merging this PR: https://holoviz-dev.github.io/examples/. You can also download an archive of the site from the workflow summary page which comes in handy when your dev site built was overriden by another PR (we have a single dev site!).

github-actions[bot] commented 1 year ago

Your changes were successfully integrated in the dev site, make sure to review the pages of the projects you touched before merging this PR: https://holoviz-dev.github.io/examples/. You can also download an archive of the site from the workflow summary page which comes in handy when your dev site built was overriden by another PR (we have a single dev site!).

github-actions[bot] commented 1 year ago

Your changes were successfully integrated in the dev site, make sure to review the pages of the projects you touched before merging this PR: https://holoviz-dev.github.io/examples/. You can also download an archive of the site from the workflow summary page which comes in handy when your dev site built was overriden by another PR (we have a single dev site!).

maximlt commented 1 year ago

To make this change we needed to trigger the driving workflow from a pull_request event and no longer a push event which means that some more changes were required.