Skip to content

chore(conventions): Bump sentry-conventions to include transform status - #6281

Merged
constantinius merged 2 commits into
masterfrom
constantinius/chore/conventions/bump-for-transform-status
Aug 3, 2026
Merged

chore(conventions): Bump sentry-conventions to include transform status#6281
constantinius merged 2 commits into
masterfrom
constantinius/chore/conventions/bump-for-transform-status

Conversation

@constantinius

Copy link
Copy Markdown
Contributor

Bumps the sentry-conventions submodule past PR #465 which adds the "transform" deprecation status for gen_ai attributes.

This is a prerequisite for #6201 which implements the gen_ai attribute transformations.

Also updates deprecated app vitals constant usages (FRAMES_FROZEN_RATEAPP__VITALS__FRAMES__FROZEN__RATE, etc.) and the Transform variant in the build script.

@constantinius
constantinius requested a review from a team as a code owner August 3, 2026 12:41
@constantinius
constantinius force-pushed the constantinius/chore/conventions/bump-for-transform-status branch from 2a3c6f0 to c961631 Compare August 3, 2026 12:42

@Dav1dde Dav1dde left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this on purpose just a bump by 3 commits?

Image

Comment thread relay-conventions/build/attributes.rs Outdated
@constantinius
constantinius force-pushed the constantinius/chore/conventions/bump-for-transform-status branch from c961631 to af19b19 Compare August 3, 2026 12:52
@constantinius

Copy link
Copy Markdown
Contributor Author

@Dav1dde I was a bit selfish and just used the commits I needed, but I've bumped to the latest conventions and it seems to be working fine.

@constantinius
constantinius force-pushed the constantinius/chore/conventions/bump-for-transform-status branch from af19b19 to e183ab4 Compare August 3, 2026 13:11
Comment thread relay-conventions/build/attributes.rs

@cursor cursor Bot 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.

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 e183ab4. Configure here.

Comment thread relay-conventions/build/attributes.rs
Bump the sentry-conventions submodule to include PR #465 which adds the
"transform" deprecation status for gen_ai attributes.

Also adds the Transform variant to DeprecationStatus in the build script
(producing WriteBehavior::CurrentName) and updates deprecated constant
usages for the app vitals attributes that were renamed in the same
range.
@constantinius
constantinius force-pushed the constantinius/chore/conventions/bump-for-transform-status branch from e183ab4 to 82f35cf Compare August 3, 2026 13:41
Comment thread CHANGELOG.md Outdated
Co-authored-by: David Herberth <david.herberth@sentry.io>
Comment thread relay-conventions/build/attributes.rs
@constantinius
constantinius added this pull request to the merge queue Aug 3, 2026
Merged via the queue into master with commit 3215baa Aug 3, 2026
38 of 54 checks passed
@constantinius
constantinius deleted the constantinius/chore/conventions/bump-for-transform-status branch August 3, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants