Skip to content

In FilterableDataInterface::withFilterHandlers() rename parameter to $filterHandlers + Fix bc.yml#137

Merged
vjik merged 2 commits intomasterfrom
fix-name
Jan 25, 2023
Merged

In FilterableDataInterface::withFilterHandlers() rename parameter to $filterHandlers + Fix bc.yml#137
vjik merged 2 commits intomasterfrom
fix-name

Conversation

@vjik
Copy link
Member

@vjik vjik commented Jan 24, 2023

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

@vjik vjik changed the title In FilterableDataInterface::withFilterHandlers() rename parameter to $filterHandlers In FilterableDataInterface::withFilterHandlers() rename parameter to $filterHandlers + Fix bc.yml Jan 24, 2023
@codecov
Copy link

codecov bot commented Jan 24, 2023

Codecov Report

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

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

Additional details and impacted files
@@             Coverage Diff             @@
##              master      #137   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity       249       249           
===========================================
  Files             35        35           
  Lines            615       615           
===========================================
  Hits             615       615           
Impacted Files Coverage Δ
src/Reader/Iterable/IterableDataReader.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 January 24, 2023 19:15
@vjik vjik added the status:code review The pull request needs review. label Jan 24, 2023
@vjik vjik merged commit 4b359d5 into master Jan 25, 2023
@vjik vjik deleted the fix-name branch January 25, 2023 14:50
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