Skip to content

Conversation

@vjik
Copy link
Member

@vjik vjik commented Apr 14, 2025

Q A
Is bugfix?
New feature?
Breaks BC? ✔️

@vjik vjik requested a review from a team April 14, 2025 13:56
@vjik vjik added the status:code review The pull request needs review. label Apr 14, 2025
@codecov
Copy link

codecov bot commented Apr 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.72%. Comparing base (05ab1c0) to head (3da42ec).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #242   +/-   ##
=========================================
  Coverage     95.72%   95.72%           
  Complexity      354      354           
=========================================
  Files            46       46           
  Lines           958      958           
=========================================
  Hits            917      917           
  Misses           41       41           

☔ 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.

@samdark samdark requested a review from viktorprogger April 14, 2025 21:36
@bautrukevich
Copy link

@viktorprogger can you look at these please? and merge 🙏

@viktorprogger
Copy link
Contributor

@bautrukevich I'll look at it this week

@viktorprogger viktorprogger merged commit 72a674d into master May 4, 2025
26 checks passed
@viktorprogger viktorprogger deleted the finalize branch May 4, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants