Skip to content

Explicitly mark nullable parameters#377

Merged
vjik merged 2 commits intomasterfrom
nulls
Feb 25, 2025
Merged

Explicitly mark nullable parameters#377
vjik merged 2 commits intomasterfrom
nulls

Conversation

@vjik
Copy link
Member

@vjik vjik commented Feb 24, 2025

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

@codecov
Copy link

codecov bot commented Feb 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.73%. Comparing base (5ef8ded) to head (05b49ac).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #377   +/-   ##
=========================================
  Coverage     98.73%   98.73%           
  Complexity      173      173           
=========================================
  Files            16       16           
  Lines           552      552           
=========================================
  Hits            545      545           
  Misses            7        7           

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

@vjik vjik added the status:code review The pull request needs review. label Feb 24, 2025
@vjik vjik requested a review from a team February 24, 2025 15:19
@vjik vjik merged commit bd8e8c2 into master Feb 25, 2025
26 checks passed
@vjik vjik deleted the nulls branch February 25, 2025 06:52
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.

3 participants