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

Docs: added new target (docs-next) to the docs' Makefile. #3962

Merged
merged 1 commit into from
Jul 8, 2021

Conversation

KMiller-Grafana
Copy link
Contributor

This target builds local docs as they should look for commits that are
placed into the main branch, showing the next up, but not yet released
Loki docs.

This target builds local docs as they should look for commits that are
placed into the main branch, showing the next up, but not yet released
Loki docs.
@KMiller-Grafana KMiller-Grafana requested review from robbymilo and a team July 7, 2021 21:17
.PHONY: docs-next
docs-next:
docker pull ${IMAGE}
docker run --rm -it -v ${PWD}/sources:/hugo/content/docs/loki/next -p 3002:3002 $(IMAGE) /bin/bash -c 'mkdir -p content/docs/grafana/next/ && touch content/docs/grafana/next/menu.yaml && make server-quick'
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity why is the touch required?

Copy link
Member

Choose a reason for hiding this comment

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

I'd guess to update the file timestamp and signal the subsequent make command that it's dependencies need to be built.

.PHONY: docs-next
docs-next:
docker pull ${IMAGE}
docker run --rm -it -v ${PWD}/sources:/hugo/content/docs/loki/next -p 3002:3002 $(IMAGE) /bin/bash -c 'mkdir -p content/docs/grafana/next/ && touch content/docs/grafana/next/menu.yaml && make server-quick'
Copy link
Member

Choose a reason for hiding this comment

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

I'd guess to update the file timestamp and signal the subsequent make command that it's dependencies need to be built.

@owen-d owen-d merged commit f653a93 into grafana:main Jul 8, 2021
@KMiller-Grafana KMiller-Grafana deleted the docs/new-make-target branch July 22, 2021 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants