Skip to content

Conversation

@mtfriesen
Copy link
Contributor

Description

Describe the purpose of and changes within this Pull Request.

Extend #5598 by rewriting commit authorship (to allow @anrossi to approve this PR) and implementing the suggested refactorings.

Testing

Do any existing tests cover this change? Are new tests needed?

Builds locally, CI.

Documentation

Is there any documentation impact for this change?

N/A.

@mtfriesen mtfriesen requested a review from a team as a code owner January 6, 2026 21:03
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.88%. Comparing base (9117f56) to head (44f6092).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5703      +/-   ##
==========================================
- Coverage   86.20%   85.88%   -0.33%     
==========================================
  Files          60       60              
  Lines       18663    18691      +28     
==========================================
- Hits        16089    16052      -37     
- Misses       2574     2639      +65     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mtfriesen mtfriesen merged commit 9724fc0 into main Jan 7, 2026
524 checks passed
@mtfriesen mtfriesen deleted the mtfriesen/object_debug branch January 7, 2026 12:50
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.

3 participants