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

Remove usage of additional Report.get arguments#1137

Merged
Swatinem merged 1 commit intomainfrom
swatinem/rm-getelse
Mar 13, 2025
Merged

Remove usage of additional Report.get arguments#1137
Swatinem merged 1 commit intomainfrom
swatinem/rm-getelse

Conversation

@Swatinem
Copy link
Copy Markdown
Contributor

I’m about to remove the additional arguments to Report.get, so lets first remove any remaining usage.

I’m about to remove the additional arguments to `Report.get`, so lets first remove any remaining usage.
@Swatinem Swatinem requested a review from a team March 13, 2025 09:56
@Swatinem Swatinem self-assigned this Mar 13, 2025
@Swatinem Swatinem enabled auto-merge March 13, 2025 10:01
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.77%. Comparing base (4a44e42) to head (98491e5).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1137      +/-   ##
==========================================
- Coverage   97.77%   97.77%   -0.01%     
==========================================
  Files         444      444              
  Lines       36637    36636       -1     
==========================================
- Hits        35821    35820       -1     
  Misses        816      816              
Flag Coverage Δ
integration 42.90% <100.00%> (-0.01%) ⬇️
unit 90.47% <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 Mar 13, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 13, 2025
@Swatinem Swatinem added this pull request to the merge queue Mar 13, 2025
Merged via the queue into main with commit b8e6f34 Mar 13, 2025
23 of 25 checks passed
@Swatinem Swatinem deleted the swatinem/rm-getelse branch March 13, 2025 11:36
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.

2 participants