Skip to content

Do not reset $retryHandler property in AbstractCommand::reset() - #1115

Merged
Tigrov merged 3 commits into
masterfrom
do-not-reset-retryHandler
Nov 30, 2025
Merged

Do not reset $retryHandler property in AbstractCommand::reset()#1115
Tigrov merged 3 commits into
masterfrom
do-not-reset-retryHandler

Conversation

@Tigrov

@Tigrov Tigrov commented Nov 30, 2025

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

Related yiisoft/yii2#19680

@Tigrov Tigrov added the type:bug Bug label Nov 30, 2025
@codecov

codecov Bot commented Nov 30, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.62%. Comparing base (a4acaee) to head (e251a7d).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1115      +/-   ##
============================================
- Coverage     98.62%   98.62%   -0.01%     
  Complexity     1655     1655              
============================================
  Files           120      120              
  Lines          4305     4304       -1     
============================================
- Hits           4246     4245       -1     
  Misses           59       59              

☔ View full report in Codecov by Sentry.
📢 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.

@Tigrov Tigrov added the status:code review The pull request needs review. label Nov 30, 2025
@Tigrov
Tigrov requested a review from a team November 30, 2025 13:30
Comment thread src/Command/AbstractCommand.php
@Tigrov
Tigrov merged commit a2fb23b into master Nov 30, 2025
69 checks passed
@Tigrov
Tigrov deleted the do-not-reset-retryHandler branch November 30, 2025 14:01
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. type:bug Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants