Skip to content

docs: describe current behavior, not change history, in comments#143

Closed
zfarrell wants to merge 1 commit into
mainfrom
chore/comment-cleanup
Closed

docs: describe current behavior, not change history, in comments#143
zfarrell wants to merge 1 commit into
mainfrom
chore/comment-cleanup

Conversation

@zfarrell

@zfarrell zfarrell commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Rewords comments that narrated the SDK-refactor history ("reproduces the old ApiClient", "verbatim from", "matching the old behavior") to describe what the code does now. Comment-only; no code or test changes.

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment-only cleanup; rewordings accurately describe current behavior and the new upload_reqwest_client intra-doc link resolves. LGTM.

@sentry

sentry Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/sdk.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@zfarrell zfarrell closed this Jun 5, 2026
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.

1 participant