Skip to content

Conversation

@KristofferFM
Copy link
Contributor

@codecov
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.77%. Comparing base (2ce84f7) to head (e5d557d).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #720   +/-   ##
=========================================
  Coverage     98.77%   98.77%           
  Complexity     1884     1884           
=========================================
  Files            71       71           
  Lines          5289     5289           
=========================================
  Hits           5224     5224           
  Misses           65       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.

@phil-davis
Copy link
Contributor

@KristofferFM could you rebase and resolve conflicts please.
There was another PR that also touched code in the same area.

@KristofferFM
Copy link
Contributor Author

@phil-davis sure, I expect to have something ready within at most a few days.

…irst upload.

As this issue has been fixed by someone else, only the test is remaining in this commit.
@KristofferFM
Copy link
Contributor Author

@phil-davis The merge conflict was due to a functionally equivalent fix of the issue by SebastianKrupinski. So that is all good :-) I have left the test in the PR, up to you if it should be pulled or not, I don't know if there are other tests covering the same area.

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.

2 participants