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

Update docs and links #39

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Update docs and links #39

merged 1 commit into from
Nov 28, 2023

Conversation

samuelallan72
Copy link
Contributor

These mainly fix things that seem to be leftovers from porting
from the grafana-agent-k8s-operator codebase.

Fixes: #16

Fixes: #21

These mainly fix things that seem to be leftovers from porting
from the grafana-agent-k8s-operator codebase.
Copy link
Contributor

@sed-i sed-i left a comment

Choose a reason for hiding this comment

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

Thanks @samuelengineer!
Need to revert the docs: changes and we're good to merge.

@@ -17,10 +17,10 @@ maintainers:
- Luca Bello <[email protected]>
- Simon Aronsson <[email protected]>

#docs: https://discourse.charmhub.io/t/grafana-agent-k8s-docs-index/5605
docs: https://github.com/canonical/grafana-agent-operator/blob/main/README.md
Copy link
Contributor

Choose a reason for hiding this comment

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

  • When there is no docs key, charmhub automatically renderes the readme.
  • Currently there are no discourse docs so the docs key should be omitted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sed-i I'm not sure that's the case any more? Currently the readme is not rendered on charmhub for grafana-agent: https://charmhub.io/grafana-agent

Copy link
Contributor

Choose a reason for hiding this comment

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

Ha, you're right. Alrighty, let's give this a try.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, and thanks for reviewing and merging! :)

@sed-i sed-i merged commit 1516e4a into canonical:main Nov 28, 2023
12 checks passed
@samuelallan72 samuelallan72 deleted the docs branch November 28, 2023 03:30
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.

--trust is too high privilege for a machine charm charmhub links point to wrong source
2 participants