Skip to content

docs: remove redundant "metrics" from data sources#9639

Merged
tiffany76 merged 1 commit intoopen-telemetry:mainfrom
sammaji:patch-1
Apr 13, 2026
Merged

docs: remove redundant "metrics" from data sources#9639
tiffany76 merged 1 commit intoopen-telemetry:mainfrom
sammaji:patch-1

Conversation

@sammaji
Copy link
Copy Markdown
Contributor

@sammaji sammaji commented Apr 13, 2026

  • I have read and followed the Contributing docs, especially the "First-time contributing?" section.
  • This PR has content that I did not fully write myself.
  • I have the experience and knowledge necessary to understand, review, and validate all content in this PR.1

Fixes a minor documentation line code comment. The code comment had duplicate word ("metrics").

Footnotes

  1. Yes, I can answer maintainer questions about the content of this PR, without using AI. ↩

@sammaji sammaji requested a review from a team as a code owner April 13, 2026 18:09
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Apr 13, 2026 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: sammaji / name: Samyabrata Maji (c400c6e)

@otelbot-docs otelbot-docs bot requested review from a team and atoulme and removed request for a team April 13, 2026 18:09
@otelbot-docs otelbot-docs bot added missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer missing:sig-approval Co-owning SIG didn't provide an approval labels Apr 13, 2026
tiffany76
tiffany76 reviewed Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

Thanks so much for submitting this PR! You're right that the comment was off, but I think the extra metrics was supposed to be traces!

Comment thread content/en/docs/collector/configuration.md Outdated
@otelbot-docs otelbot-docs bot requested a review from a team April 13, 2026 19:17
tiffany76
tiffany76 approved these changes Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

Thanks again!

@tiffany76 tiffany76 added this pull request to the merge queue Apr 13, 2026
Merged via the queue into open-telemetry:main with commit 81dc000 Apr 13, 2026
25 checks passed
@otelbot
Copy link
Copy Markdown
Contributor

otelbot bot commented Apr 13, 2026

Thank you for your contribution @sammaji! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

@sammaji sammaji deleted the patch-1 branch April 13, 2026 20:02
tiffany76 pushed a commit to tiffany76/opentelemetry.io that referenced this pull request Apr 13, 2026
@sammaji @tiffany76
docs: remove redundant "metrics" from data sources (open-telemetry#9639)
14be4f4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tiffany76 tiffany76 tiffany76 approved these changes

@atoulme atoulme Awaiting requested review from atoulme atoulme was automatically assigned from open-telemetry/collector-approvers

Assignees

No one assigned

Labels

missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer missing:sig-approval Co-owning SIG didn't provide an approval sig:collector

Projects

SIG Comms: PRs & Issues
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sammaji @tiffany76
⚡