Skip to content

fix(rq): Restore functools.wraps() for patched functions - #6532

Merged
alexander-alderman-webb merged 1 commit into
masterfrom
webb/rq/add-wraps
Jun 8, 2026
Merged

fix(rq): Restore functools.wraps() for patched functions#6532
alexander-alderman-webb merged 1 commit into
masterfrom
webb/rq/add-wraps

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

Fix regression introduced in #6493 because I didn't realize ensure_integration_enabled also wraps the patched function.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code ownerJune 8, 2026 12:31
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

88776 passed | ⏭️ 6025 skipped | Total: 94801 | Pass Rate: 93.64% | Execution Time: 302m 39s

📊 Comparison with Base Branch

MetricChange
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2355 uncovered lines.
✅ Project coverage is 89.9%. Comparing base (base) to head (head).

Coverage diff
@@ Coverage Diff @@## main #PR +/-##
==========================================
+ Coverage 89.90% 89.90% —%
==========================================
Files 192 192 —
Lines 23321 23324 +3
Branches 8020 8020 —
==========================================
+ Hits 20967 20969 +2- Misses 2354 2355 +1- Partials 1326 1327 +1

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb merged commit afff0e2 into masterJun 8, 2026
144 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/rq/add-wraps branch June 8, 2026 12:45
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
…#6532)
Fix regression from getsentry@3d126ff by adding `functools.wraps()` where `ensure_integration_enabled()` was dropped.
Sign up for freeto 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.

2 participants

@alexander-alderman-webb@sentrivana