Skip to content

ref: Rename set_span_data to set_span_attribute #92946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 5, 2025

Conversation

szokeasaurusrex
Copy link
Member

sentry_sdk.Span.set_data will be replaced with sentry_sdk.Span.set_attribute in Sentry SDK 3.0.

This change renames the set_span_data method to set_span_attribute to match the new method name in the SDK.

Split off from #92011

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

`with sentry_sdk.init` is deprecated and will be removed in Sentry SDK 3.0.

Split off from #92011
`sentry_sdk.Span.set_data` will be replaced with `sentry_sdk.Span.set_attribute` in Sentry SDK 3.0.

This change renames the `set_span_data` method to `set_span_attribute` to match the new method name in the SDK.

Split off from #92011
@szokeasaurusrex szokeasaurusrex requested review from a team as code owners June 5, 2025 14:57
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 5, 2025
Copy link

codecov bot commented Jun 5, 2025

Codecov Report

Attention: Patch coverage is 92.15686% with 4 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/search/events/fields.py 25.00% 3 Missing ⚠️
.../sentry/api/endpoints/organization_events_trace.py 85.71% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #92946   +/-   ##
=======================================
  Coverage   87.89%   87.90%           
=======================================
  Files       10284    10284           
  Lines      590365   590316   -49     
  Branches    22961    22947   -14     
=======================================
- Hits       518917   518895   -22     
+ Misses      71001    70974   -27     
  Partials      447      447           

Copy link
Member

@asottile-sentry asottile-sentry left a comment

Choose a reason for hiding this comment

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

Base automatically changed from szokeasaurusrex/set-client to master June 5, 2025 15:24
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) June 5, 2025 17:05
@szokeasaurusrex szokeasaurusrex merged commit 18ed36d into master Jun 5, 2025
60 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/set_span_attribute branch June 5, 2025 17:25
@github-actions github-actions bot locked and limited conversation to collaborators Jun 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants