Skip to content

Fix typos in sdk-common Javadoc#8512

Merged
jack-berg merged 1 commit into
open-telemetry:mainfrom
thswlsqls:docs/fix-sdk-common-javadoc-typos
Jul 13, 2026
Merged

Fix typos in sdk-common Javadoc#8512
jack-berg merged 1 commit into
open-telemetry:mainfrom
thswlsqls:docs/fix-sdk-common-javadoc-typos

Conversation

@thswlsqls

Copy link
Copy Markdown
Contributor

Description

  • Fix five Javadoc typos in the sdk-common module; text-only, no behavior or signature change.
  • InternalTelemetryVersion: "prior the standardization" -> "prior to the standardization".
  • Resource: "which capture identifying information" -> "which captures" (singular subject).
  • RandomSupplier: "random number generater" -> "generator".
  • DefaultExceptionAttributeResolver: drop stray () after the @see ...getDefault(boolean) reference.
  • AndroidFriendlyRandomHolder: "everytime" -> "every time".

Testing done

  • Docs-only, test-exempt: no test added; ./gradlew :sdk:common:check passed (compile, spotless, ErrorProne/NullAway, jApiCmp).
  • No signature change, so jApiCmp produced no apidiff; not user-facing, so no CHANGELOG entry.

@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.77%. Comparing base (824334c) to head (41a111a).
⚠️ Report is 59 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8512   +/-   ##
=========================================
  Coverage     78.77%   78.77%           
  Complexity     8579     8579           
=========================================
  Files          1009     1009           
  Lines         28993    28993           
  Branches       3599     3599           
=========================================
  Hits          22839    22839           
  Misses         5311     5311           
  Partials        843      843           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jack-berg

Copy link
Copy Markdown
Member

Hey @thswlsqls are you planning on picking back up and continuing any of these PRs you have open?

I'm not reviewing them while their in draft state. That's sort of an unwritten rule of the project.

If you no longer have time, consider just marking them all ready for review in bulk. I can merge, or close, or push minor changes and merge as needed.

Thanks

@jack-berg jack-berg added the needs author feedback Waiting for additional feedback from the author label Jul 2, 2026
@thswlsqls

Copy link
Copy Markdown
Contributor Author

Thanks @jack-berg — I'll mark them ready for review in bulk. Please merge, close, or push minor fixes as you see fit.

@github-actions github-actions Bot removed the needs author feedback Waiting for additional feedback from the author label Jul 4, 2026
@jack-berg
jack-berg marked this pull request as ready for review July 13, 2026 19:43
@jack-berg
jack-berg requested a review from a team as a code owner July 13, 2026 19:43
@jack-berg
jack-berg merged commit 6e85945 into open-telemetry:main Jul 13, 2026
29 checks passed
@otelbot

otelbot Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

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

@thswlsqls
thswlsqls deleted the docs/fix-sdk-common-javadoc-typos branch July 17, 2026 02:31
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.

2 participants