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 make docs procedure #472

Merged
merged 3 commits into from
Jul 1, 2024

Conversation

jdbaldry
Copy link
Member

Add the tooling for rendering a local preview of the documentation and a procedure to keep it up to date.

For more information, refer to https://grafana.com/docs/writers-toolkit/review/.

Signed-off-by: Jack Baldry jack.baldry@grafana.com

Add the tooling for rendering a local preview of the documentation.

For more information, refer to https://grafana.com/docs/writers-toolkit/review/.

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>
Signed-off-by: Jack Baldry <jack.baldry@grafana.com>
Copy link
Collaborator

@JStickler JStickler left a comment

Choose a reason for hiding this comment

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

[docs team] LGTM.

Do we need anything in the way of workflows to publish to the website?

@jdbaldry
Copy link
Member Author

For publishing "next" content, we first need to merge https://github.com/grafana/website/pull/20393 to prevent publishing prematurely and then I an install something like https://github.com/grafana/grafana/blob/main/.github/workflows/publish-technical-documentation-next.yml to do the publishing.

Do we have an issue somewhere that explains where we want the docs to show up in the website?

@JStickler
Copy link
Collaborator

Do we have an issue somewhere that explains where we want the docs to show up in the website?

We're having an information architecture meeting on Tuesday to figure that out.

@jdbaldry
Copy link
Member Author

jdbaldry commented Jul 1, 2024

#494 was merged that will sync documentation

Copy link
Collaborator

@matryer matryer left a comment

Choose a reason for hiding this comment

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

LGTM

@matryer matryer merged commit b6b84e8 into grafana:main Jul 1, 2024
3 checks passed
@jdbaldry jdbaldry deleted the 2024-06-add-make-docs-procedure branch July 3, 2024 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants