Skip to content

Allow use AtLeast rule for arrays + tests refactoring#334

Merged
vjik merged 2 commits into
masterfrom
fix-atleast
Oct 12, 2022
Merged

Allow use AtLeast rule for arrays + tests refactoring#334
vjik merged 2 commits into
masterfrom
fix-atleast

Conversation

@vjik

@vjik vjik commented Oct 11, 2022

Copy link
Copy Markdown
Member
Q A
Is bugfix? ✔️
New feature? ✔️
Breaks BC?
Fixed issues -

@codecov

codecov Bot commented Oct 11, 2022

Copy link
Copy Markdown

Codecov Report

Base: 95.79% // Head: 95.79% // No change to project coverage 👍

Coverage data is based on head (eea95ce) compared to base (fddb9c6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #334   +/-   ##
=========================================
  Coverage     95.79%   95.79%           
  Complexity      622      622           
=========================================
  Files            73       73           
  Lines          1476     1476           
=========================================
  Hits           1414     1414           
  Misses           62       62           
Impacted Files Coverage Δ
src/Rule/AtLeastHandler.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 vjik requested a review from a team October 11, 2022 13:26
@vjik vjik added the status:code review The pull request needs review. label Oct 11, 2022
@vjik vjik merged commit 190ddda into master Oct 12, 2022
@vjik vjik deleted the fix-atleast branch October 12, 2022 05:18
@vjik vjik mentioned this pull request Oct 12, 2022
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.

4 participants