Skip to content

chore(dynamic-sampling): rename dynamic sampling status enum#115360

Merged
shellmayr merged 2 commits into
masterfrom
shellmayr/chore/rename-ds-status-enum
May 18, 2026
Merged

chore(dynamic-sampling): rename dynamic sampling status enum#115360
shellmayr merged 2 commits into
masterfrom
shellmayr/chore/rename-ds-status-enum

Conversation

@shellmayr
Copy link
Copy Markdown
Member

This name makes more sense - it's the status of the dynamic sampling job, so let's not make this more complicated than necessary

@shellmayr shellmayr requested a review from a team as a code owner May 12, 2026 07:16
@shellmayr shellmayr changed the title Shellmayr/chore/rename ds status enum chore(dynamic-sampling): rename dynamic sampling status enum May 12, 2026
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 12, 2026
Comment thread src/sentry/dynamic_sampling/per_org/tasks/telemetry.py
Comment thread src/sentry/dynamic_sampling/per_org/tasks/telemetry.py Outdated
@shellmayr shellmayr force-pushed the shellmayr/chore/rename-ds-status-enum branch from caabae7 to d6b63cc Compare May 18, 2026 11:17
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d6b63cc. Configure here.

Comment thread src/sentry/dynamic_sampling/per_org/tasks/telemetry.py
Comment thread src/sentry/dynamic_sampling/per_org/tasks/telemetry.py Outdated
@shellmayr shellmayr merged commit 6dda0a7 into master May 18, 2026
62 checks passed
@shellmayr shellmayr deleted the shellmayr/chore/rename-ds-status-enum branch May 18, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants