Skip to content
This repository was archived by the owner on Jun 13, 2025. It is now read-only.

Conversation

@Swatinem
Copy link
Contributor

This will now return an empty response with a deprecation message, and will not call out to the worker task anymore.

@seer-by-sentry
Copy link
Contributor

seer-by-sentry bot commented Apr 17, 2025

🚨 Sentry detected 2 potential issues in your recent changes 🚨

Did you find this useful? React with a 👍 or 👎

@Swatinem Swatinem force-pushed the swatinem/deprecate-reportresults branch 2 times, most recently from 65bd8ce to 7b14a29 Compare April 17, 2025 08:47
@Swatinem Swatinem marked this pull request as ready for review April 23, 2025 11:46
@Swatinem Swatinem requested a review from a team as a code owner April 23, 2025 11:46
This will now return an empty response with a deprecation message, and will not call out to the worker task anymore.
@Swatinem Swatinem force-pushed the swatinem/deprecate-reportresults branch from 7b14a29 to de8f160 Compare April 25, 2025 07:25
@codecov-notifications
Copy link

codecov-notifications bot commented Apr 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Apr 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.43%. Comparing base (6546390) to head (de8f160).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1291      +/-   ##
==========================================
- Coverage   96.43%   96.43%   -0.01%     
==========================================
  Files         488      488              
  Lines       17006    16975      -31     
==========================================
- Hits        16400    16370      -30     
+ Misses        606      605       -1     
Flag Coverage Δ
unit 96.43% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@Swatinem Swatinem added this pull request to the merge queue Apr 25, 2025
Merged via the queue into main with commit 2ead8c6 Apr 25, 2025
24 of 25 checks passed
@Swatinem Swatinem deleted the swatinem/deprecate-reportresults branch April 25, 2025 07:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants