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

Conversation

@Swatinem
Copy link
Contributor

The Sentry message I introduced was never hit, so it is fair to say that we are not running into any incompatibility there.

The Sentry message I introduced was never hit, so it is fair to say that we are not running into any incompatibility there.
@Swatinem Swatinem requested a review from a team April 16, 2025 08:39
@Swatinem Swatinem self-assigned this Apr 16, 2025
@codecov
Copy link

codecov bot commented Apr 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.59%. Comparing base (9401035) to head (32a0b9a).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #604      +/-   ##
==========================================
- Coverage   88.60%   88.59%   -0.01%     
==========================================
  Files         463      463              
  Lines       12773    12763      -10     
  Branches     1455     1453       -2     
==========================================
- Hits        11317    11307      -10     
  Misses       1151     1151              
  Partials      305      305              
Flag Coverage Δ
shared-docker-uploader 88.59% <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.

@codecov-notifications
Copy link

codecov-notifications bot commented Apr 16, 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!

@codspeed-hq
Copy link

codspeed-hq bot commented Apr 16, 2025

CodSpeed Performance Report

Merging #604 will not alter performance

Comparing swatinem/no-flagpatterncompat (32a0b9a) with main (9401035)

Summary

✅ 9 untouched benchmarks

@Swatinem Swatinem enabled auto-merge April 16, 2025 09:20
@Swatinem Swatinem added this pull request to the merge queue Apr 16, 2025
Merged via the queue into main with commit 133c177 Apr 16, 2025
12 checks passed
@Swatinem Swatinem deleted the swatinem/no-flagpatterncompat branch April 16, 2025 13: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