Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor normalizeDefaultValue() #363

Merged
merged 4 commits into from
Nov 16, 2024
Merged

Conversation

Tigrov
Copy link
Member

@Tigrov Tigrov commented Nov 8, 2024

Related PR

Q A
Is bugfix? ✔️/❌
New feature?
Breaks BC?
Fixed issues

Copy link

codecov bot commented Nov 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.91%. Comparing base (b8ffabc) to head (09def87).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #363      +/-   ##
============================================
+ Coverage     98.90%   98.91%   +0.01%     
- Complexity      171      174       +3     
============================================
  Files            17       17              
  Lines           547      554       +7     
============================================
+ Hits            541      548       +7     
  Misses            6        6              

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

@Tigrov Tigrov requested a review from a team November 12, 2024 06:47
@Tigrov Tigrov merged commit 7a74430 into master Nov 16, 2024
2 checks passed
@Tigrov Tigrov deleted the refactor-normalize-default-value branch November 16, 2024 02:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants