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 deploy preview workflow #10324

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

add deploy preview workflow #10324

wants to merge 1 commit into from

Conversation

robbymilo
Copy link
Contributor

for https://github.com/grafana/website/issues/10101

What is this feature?

This PR adds a workflow to deploy the output of the make docs command to a public deploy preview. The workflow posts a comment with a link to the preview after it has been deployed.

Why do we need this feature?

Docs contributors will no longer need to run make docs on their machine, as they will be able to view the deploy preview instead.

Who is this feature for?

Everyone.

@robbymilo robbymilo requested a review from a team as a code owner December 30, 2024 10:25
@CLAassistant
Copy link

CLAassistant commented Dec 30, 2024

CLA assistant check
All committers have signed the CLA.

@robbymilo robbymilo requested a review from a team December 30, 2024 10:26
@tacole02
Copy link
Contributor

❤️ 🥳 🎆

Comment on lines +22 to +23
website_directory: content/docs/mimir/latest
relative_prefix: /docs/mimir/latest/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this repo also hosts the mimir-distributed helm chart docs under https://github.com/grafana/mimir/tree/3050e72641187ecaf84792036d23843d37f8cbce/docs/sources/helm-charts/mimir-distributed

does this config prevent from rendering those docs too?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well remembered, I think we'll need to modify and release a v2 of the workflow that supports multiple source directories.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the mean time

Suggested change
website_directory: content/docs/mimir/latest
relative_prefix: /docs/mimir/latest/
source_directory: docs/sources/mimir
website_directory: content/docs/mimir/latest
relative_prefix: /docs/mimir/latest/

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.

5 participants