Skip to content

Fix typos in sdk-common Javadoc#8512

Draft
thswlsqls wants to merge 1 commit into
open-telemetry:mainfrom
thswlsqls:docs/fix-sdk-common-javadoc-typos
Draft

Fix typos in sdk-common Javadoc#8512
thswlsqls wants to merge 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 40 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs author feedback Waiting for additional feedback from the author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants