Skip to content

Conversation

olegbaturin
Copy link
Contributor

Q A
Is bugfix? ✔️
New feature?
Breaks BC?

Removes duplicated Stack trace block in the error message.

Copy link

codecov bot commented Sep 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.09%. Comparing base (033468a) to head (d70a559).
Report is 39 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #130      +/-   ##
============================================
- Coverage     78.86%   77.09%   -1.78%     
- Complexity      170      179       +9     
============================================
  Files            13       13              
  Lines           511      550      +39     
============================================
+ Hits            403      424      +21     
- Misses          108      126      +18     

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

Copy link
Member

@vjik vjik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to add line to changelog

@vjik vjik requested a review from a team September 17, 2024 14:09
@vjik vjik added the status:code review The pull request needs review. label Sep 17, 2024
@samdark samdark requested a review from xepozz September 17, 2024 14:28
update changelog
@vjik vjik merged commit 205ba26 into yiisoft:master Sep 19, 2024
16 of 19 checks passed
@olegbaturin olegbaturin deleted the update-logger-message branch September 19, 2024 07: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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants