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

Doc: fix broken links in production/README.md #2702

Merged
merged 1 commit into from
Oct 12, 2020

Conversation

huikang
Copy link
Contributor

@huikang huikang commented Sep 30, 2020

Signed-off-by: Hui Kang kangh@us.ibm.com

@CLAassistant
Copy link

CLAassistant commented Sep 30, 2020

CLA assistant check
All committers have signed the CLA.

@codecov-commenter
Copy link

codecov-commenter commented Sep 30, 2020

Codecov Report

Merging #2702 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2702      +/-   ##
==========================================
- Coverage   61.41%   61.33%   -0.09%     
==========================================
  Files         173      173              
  Lines       13444    13444              
==========================================
- Hits         8257     8246      -11     
- Misses       4430     4443      +13     
+ Partials      757      755       -2     
Impacted Files Coverage Δ
pkg/promtail/positions/positions.go 46.80% <0.00%> (-11.71%) ⬇️

@slim-bean
Copy link
Collaborator

Thanks for the PR @huikang! we love it when people improve errors in our docs ❤️

Our doc situation got a little bit screwed up when we moved the docs to https://grafana.com/docs/loki

We have been trying to remove all links which directly reference the github contents as these will not render very well anymore now that we have hugo frontmatter and they are no longer named README.

Could you update these links instead to be links to the grafana.com docs?

e.g. https://grafana.com/docs/loki/latest/logql/ and https://grafana.com/docs/loki/latest/getting-started/grafana/

Thanks!

- point to the grafana/loki doc website

Signed-off-by: Hui Kang <kangh@us.ibm.com>
Copy link
Contributor

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM

@cyriltovena cyriltovena merged commit 92ace83 into grafana:master Oct 12, 2020
cyriltovena pushed a commit to cyriltovena/loki that referenced this pull request Oct 21, 2020
- point to the grafana/loki doc website

Signed-off-by: Hui Kang <kangh@us.ibm.com>
Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
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.

None yet

5 participants