Skip to content

docs(sdk): fix typos in SpanLimits docstring - #5658

Open
Dotify71 wants to merge 5 commits into
open-telemetry:mainfrom
Dotify71:fix-span-limits-docstring-typos
Open

Dotify71 wants to merge 5 commits into
open-telemetry:mainfrom
Dotify71:fix-span-limits-docstring-typos

Conversation

@Dotify71

Copy link
Copy Markdown

Description

Fixes #5657

Fixes typos and grammatical errors in the SpanLimits class docstring in opentelemetry-sdk:

  • Changed "be enforce" to "be enforced".
  • Changed "an a way read" to "a way to read".

Type of change

  • This change requires a documentation update

How Has This Been Tested?

  • Verified docstring formatting.

Does This PR Require a Contrib Repo Change?

  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Documentation has been updated

@Dotify71
Dotify71 requested a review from a team as a code owner September 13, 2026 04:29
@opentelemetry-pr-dashboard

Copy link
Copy Markdown

Pull request dashboard status

Waiting on reviewers · refreshed 2026-09-13 22:38 UTC

Review the latest changes.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

Comment thread .changelog/5657.changed
@Dotify71
Dotify71 force-pushed the fix-span-limits-docstring-typos branch from e3bd17b to d61c41f Compare September 15, 2026 02:50
@Dotify71
Dotify71 force-pushed the fix-span-limits-docstring-typos branch from 4233a47 to 38106be Compare September 15, 2026 16:26
@github-project-automation github-project-automation Bot moved this to Approved PRs in Python PR digest Sep 15, 2026

@Shriprasad-P Shriprasad-P left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Solid direction. Worth double-checking backward compatibility for existing callers of the touched API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved PRs

Development

Successfully merging this pull request may close these issues.

[docs] Fix typos and grammatical errors in SpanLimits docstring

4 participants