Skip to content

Applying Yii2 coding standards#287

Merged
terabytesoftw merged 3 commits intoyiisoft:masterfrom
s1lver:coding_standart
Jan 24, 2026
Merged

Applying Yii2 coding standards#287
terabytesoftw merged 3 commits intoyiisoft:masterfrom
s1lver:coding_standart

Conversation

@s1lver
Copy link
Copy Markdown
Member

@s1lver s1lver commented Jan 24, 2026

Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 24, 2026

Codecov Report

❌ Patch coverage is 46.87500% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.88%. Comparing base (c9863aa) to head (0e58fee).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/Predis/Command/CommandDecorator.php 30.00% 14 Missing ⚠️
src/LuaScriptBuilder.php 80.00% 2 Missing ⚠️
src/Connection.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #287      +/-   ##
============================================
- Coverage     81.45%   80.88%   -0.57%     
  Complexity      368      368              
============================================
  Files            10       10              
  Lines           895      905      +10     
============================================
+ Hits            729      732       +3     
- Misses          166      173       +7     

☔ 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.

@s1lver s1lver marked this pull request as ready for review January 24, 2026 13:30
@terabytesoftw terabytesoftw added the status:code review The pull request needs review. label Jan 24, 2026
@terabytesoftw terabytesoftw merged commit d35dba3 into yiisoft:master Jan 24, 2026
13 of 15 checks passed
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