Skip to content

ref: bump relay to 0.9.12 #97818

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Aug 14, 2025
Merged

ref: bump relay to 0.9.12 #97818

merged 2 commits into from
Aug 14, 2025

Conversation

@brendanhsentry
Copy link
Member

@brendanhsentry brendanhsentry commented Aug 13, 2025

Adds Prevent data categories from getsentry/relay#5052

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 13, 2025
@brendanhsentry brendanhsentry requested a review from a team August 13, 2025 22:39
@brendanhsentry brendanhsentry marked this pull request as ready for review August 13, 2025 22:39
@brendanhsentry brendanhsentry requested a review from a team as a code owner August 13, 2025 22:39
@brendanhsentry
Copy link
Member Author

brendanhsentry commented Aug 13, 2025

@joshuarli is this the right way to update versions with uv? Lmk if I'm missing something

cursor[bot]

This comment was marked as outdated.

Sign in to view

joshuarli
joshuarli approved these changes Aug 13, 2025
View reviewed changes
Copy link
Member

@joshuarli joshuarli left a comment

Choose a reason for hiding this comment

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

Oh hmm instead of uv lock it needs to be uv lock --upgrade-package sentry-relay to actually update, i've done this for you but need to look into this

@joshuarli
Copy link
Member

joshuarli commented Aug 13, 2025

Oh hmm instead of uv lock it needs to be uv lock --upgrade-package sentry-relay to actually update, i've done this for you but need to look into this

@asottile-sentry uv lock surprisingly doesn't do anything after sentry-relay>=0.9.11 -> sentry-relay>=0.9.12 (bug? clearly the lockfile isn't satisfying constraints in pyproject.toml anymore), it needs to be explicitly told to upgrade that one package: uv lock --upgrade-package sentry-relay. Interestingly enough, uv lock --upgrade-package foobar also... works but also feels like a bug I might report. Should we.... switch back to == for top level? sentry-relay==0.9.11 -> sentry-relay==0.9.12 then uv lock works as expected.

Related discussion: astral-sh/uv#6692 (comment)

@asottile-sentry
Copy link
Contributor

asottile-sentry commented Aug 14, 2025

god uv sucks. why would it work like that

we should not use ==

@joshuarli joshuarli merged commit bb27442 into master Aug 14, 2025
63 checks passed
@joshuarli joshuarli deleted the brendan/bump-sentry-relay branch August 14, 2025 18:04
@joshuarli
Copy link
Member

joshuarli commented Aug 14, 2025

Huh wtf I can't reproduce anymore, uv lock works now

priscilawebdev pushed a commit that referenced this pull request Aug 25, 2025
@brendanhsentry @priscilawebdev
ref: bump relay to 0.9.12 (#97818)
25cea83
andrewshie-sentry pushed a commit that referenced this pull request Aug 26, 2025
@brendanhsentry @andrewshie-sentry
ref: bump relay to 0.9.12 (#97818)
c8fda66
@github-actions github-actions bot locked and limited conversation to collaborators Aug 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

@vbro vbro vbro approved these changes

@joshuarli joshuarli joshuarli approved these changes

Assignees

No one assigned

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@brendanhsentry @joshuarli @asottile-sentry @vbro