Skip to content

Remove withMiddlewares() and withMiddlewaresAdded() methods from QueueInterface#316

Merged
samdark merged 6 commits into
masterfrom
rm-with-middleware
Jul 2, 2026
Merged

Remove withMiddlewares() and withMiddlewaresAdded() methods from QueueInterface#316
samdark merged 6 commits into
masterfrom
rm-with-middleware

Conversation

@vjik

@vjik vjik commented Jul 2, 2026

Copy link
Copy Markdown
Member
Q A
Is bugfix?
New feature?
Breaks BC? ✔️
Tests pass? ✔️

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (ccc4368) to head (cbb2159).

Files with missing lines Patch % Lines
src/Queue.php 0.00% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             master    #316   +/-   ##
========================================
  Coverage      0.00%   0.00%           
+ Complexity      329     324    -5     
========================================
  Files            51      51           
  Lines           923     906   -17     
========================================
+ Misses          923     906   -17     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vjik vjik requested a review from a team July 2, 2026 08:05
@vjik vjik added the status:code review The pull request needs review. label Jul 2, 2026
@samdark samdark merged commit 7e141c5 into master Jul 2, 2026
28 of 29 checks passed
@samdark samdark deleted the rm-with-middleware branch July 2, 2026 17:08
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.

2 participants