Skip to content

Add Message::trace() + Minor refactoring#113

Merged
vjik merged 4 commits intomasterfrom
improve-trace
Jun 22, 2024
Merged

Add Message::trace() + Minor refactoring#113
vjik merged 4 commits intomasterfrom
improve-trace

Conversation

@vjik
Copy link
Copy Markdown
Member

@vjik vjik commented Jun 22, 2024

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

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.55%. Comparing base (c53b9a9) to head (f1180b8).

Current head f1180b8 differs from pull request most recent head 0514c5c

Please upload reports for the commit 0514c5c to get more accurate results.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #113   +/-   ##
=========================================
  Coverage     99.54%   99.55%           
  Complexity      161      161           
=========================================
  Files            10       10           
  Lines           444      446    +2     
=========================================
+ Hits            442      444    +2     
  Misses            2        2           

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

@vjik vjik requested a review from a team June 22, 2024 09:40
@vjik vjik added the status:code review The pull request needs review. label Jun 22, 2024
Comment thread src/Message.php Outdated
Co-authored-by: Alexander Makarov <sam@rmcreative.ru>
@vjik vjik merged commit 447337f into master Jun 22, 2024
@vjik vjik deleted the improve-trace branch June 22, 2024 15:51
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