Skip to content

fix!(sentry-log): Make RecordMapping non-exhaustive#1098

Merged
szokeasaurusrex merged 1 commit intomasterfrom
szokeasaurusrex/record-mapping-non-exhaustive
Apr 27, 2026
Merged

fix!(sentry-log): Make RecordMapping non-exhaustive#1098
szokeasaurusrex merged 1 commit intomasterfrom
szokeasaurusrex/record-mapping-non-exhaustive

Conversation

@szokeasaurusrex
Copy link
Copy Markdown
Member

Make RecordMapping non-exhaustive, so that enabling the log feature is non-breaking.

BREAKING CHANGE: sentry_log::RecordMapping is now non-exhaustive

Closes #1089
Closes RUST-202

Make `RecordMapping` non-exhaustive, so that enabling the log feature is non-breaking.

BREAKING CHANGE: `sentry_log::RecordMapping` is now non-exhaustive

Closes #1089
Closes [RUST-202](https://linear.app/getsentry/issue/RUST-202/sentry-log-add-non-exhaustive-to-recordmapping)
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Apr 27, 2026

@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review April 27, 2026 16:25
@szokeasaurusrex szokeasaurusrex requested a review from lcian as a code owner April 27, 2026 16:25
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) April 27, 2026 16:25
@szokeasaurusrex szokeasaurusrex merged commit 84b31c4 into master Apr 27, 2026
26 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/record-mapping-non-exhaustive branch April 27, 2026 16:40
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.

sentry-log: add #[non_exhaustive] to RecordMapping

2 participants