Skip to content

fix(ratelimit): Fix clippy::arithmetic_side_effects lint#1121

Merged
szokeasaurusrex merged 1 commit into
masterfrom
szokeasaurusrex/checked-arithmetic-ratelimit
May 7, 2026
Merged

fix(ratelimit): Fix clippy::arithmetic_side_effects lint#1121
szokeasaurusrex merged 1 commit into
masterfrom
szokeasaurusrex/checked-arithmetic-ratelimit

Conversation

@szokeasaurusrex
Copy link
Copy Markdown
Member

@szokeasaurusrex szokeasaurusrex commented May 6, 2026

Addresses lint violations in sentry/src/transports/ratelimit.rs.

Stacked on #1119

Closes #1115
Closes RUST-212

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 6, 2026

@szokeasaurusrex szokeasaurusrex requested a review from giortzisg May 6, 2026 16:37
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review May 6, 2026 16:37
@szokeasaurusrex szokeasaurusrex requested a review from lcian as a code owner May 6, 2026 16:37
Copy link
Copy Markdown
Member

@lcian lcian left a comment

Choose a reason for hiding this comment

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

lgtm

Base automatically changed from szokeasaurusrex/checked-arithmetic to master May 7, 2026 11:49
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/checked-arithmetic-ratelimit branch from cee595c to e87962b Compare May 7, 2026 11:50
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) May 7, 2026 11:50
Copy link
Copy Markdown

@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 e87962b. Configure here.

Comment thread sentry/src/transports/ratelimit.rs Outdated
Comment thread sentry/src/transports/ratelimit.rs Outdated
@szokeasaurusrex szokeasaurusrex disabled auto-merge May 7, 2026 11:52
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/checked-arithmetic-ratelimit branch from e87962b to 222889c Compare May 7, 2026 11:53
@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) May 7, 2026 11:54
@szokeasaurusrex szokeasaurusrex merged commit df88275 into master May 7, 2026
24 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/checked-arithmetic-ratelimit branch May 7, 2026 11:58
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.

Fix checked arithmetic in sentry/src/transports/ratelimit.rs

2 participants