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] Update definition for traces #3060

Merged
merged 2 commits into from
Nov 17, 2023
Merged

Conversation

knylander-grafana
Copy link
Contributor

What this PR does:

Incorporates changes from PR #2694 (author never signed CLA)

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

…3061)

* Use glossary lookups instead of duplicating that information

Signed-off-by: Jack Baldry <[email protected]>

* Fix `make-docs` script typo

Signed-off-by: Jack Baldry <[email protected]>

---------

Signed-off-by: Jack Baldry <[email protected]>
@knylander-grafana
Copy link
Contributor Author

Closing this PR. The definition has been included in the website repo in glossary.yaml. this definition is now modular and will be used consistently across the site.

@knylander-grafana
Copy link
Contributor Author

I need to merge Jack's changes so I reopened this PR. @jdbaldry would you review?

@knylander-grafana knylander-grafana changed the title Update definition for traces [DOC] Update definition for traces Nov 9, 2023
Copy link
Member

@jdbaldry jdbaldry left a comment

Choose a reason for hiding this comment

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

LGTM!

@knylander-grafana knylander-grafana merged commit 44761ce into main Nov 17, 2023
16 checks passed
@knylander-grafana knylander-grafana deleted the knylander-grafana-patch-1 branch November 17, 2023 22:41
github-actions bot pushed a commit that referenced this pull request Nov 17, 2023
* Update definition for traces

* [DOC] Use glossary lookups instead of duplicating that information  (#3061)

* Use glossary lookups instead of duplicating that information

Signed-off-by: Jack Baldry <[email protected]>

* Fix `make-docs` script typo

Signed-off-by: Jack Baldry <[email protected]>

---------

Signed-off-by: Jack Baldry <[email protected]>

---------

Signed-off-by: Jack Baldry <[email protected]>
Co-authored-by: Jack Baldry <[email protected]>
(cherry picked from commit 44761ce)
knylander-grafana added a commit that referenced this pull request Nov 22, 2023
* Update definition for traces

* [DOC] Use glossary lookups instead of duplicating that information  (#3061)

* Use glossary lookups instead of duplicating that information

Signed-off-by: Jack Baldry <[email protected]>

* Fix `make-docs` script typo

Signed-off-by: Jack Baldry <[email protected]>

---------

Signed-off-by: Jack Baldry <[email protected]>

---------

Signed-off-by: Jack Baldry <[email protected]>
Co-authored-by: Jack Baldry <[email protected]>
(cherry picked from commit 44761ce)

Co-authored-by: Kim Nylander <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release-v2.3 type/docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants