Skip to content

Conversation

@vjik
Copy link
Member

@vjik vjik commented Nov 2, 2022

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

@what-the-diff
Copy link

what-the-diff bot commented Nov 2, 2022

@vjik vjik requested a review from a team November 2, 2022 06:44
@vjik vjik added the status:code review The pull request needs review. label Nov 2, 2022
@codecov
Copy link

codecov bot commented Nov 2, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@bf5bc2a). Click here to learn what that means.
Patch coverage: 100.00% of modified lines in pull request are covered.

❗ Current head b750889 differs from pull request most recent head 3471532. Consider uploading reports for the commit 3471532 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             master       #48   +/-   ##
==========================================
  Coverage          ?   100.00%           
  Complexity        ?       232           
==========================================
  Files             ?        14           
  Lines             ?       528           
  Branches          ?         0           
==========================================
  Hits              ?       528           
  Misses            ?         0           
  Partials          ?         0           
Impacted Files Coverage Δ
src/Helpers/DefinitionValidator.php 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

vjik and others added 2 commits November 3, 2022 14:06
Co-authored-by: Alexander Makarov <sam@rmcreative.ru>
@vjik vjik merged commit 35f4774 into master Nov 3, 2022
@vjik vjik deleted the fix branch November 3, 2022 11:10
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