Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mutex #119

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Mutex #119

wants to merge 3 commits into from

Conversation

xepozz
Copy link
Member

@xepozz xepozz commented Jun 18, 2023

Q A
Is bugfix? ✔️/❌
New feature? ✔️/❌
Breaks BC? ✔️/❌
Fixed issues comma-separated list of tickets # fixed by the PR, if any

@what-the-diff
Copy link

what-the-diff bot commented Jun 18, 2023

PR Summary

  • New dependency added
    A new dependency was included in the project
  • Updated DebugController
    Improved the way DebugController handles certain tasks by changing the components it depends on and modifying its methods
  • Enhanced debug entry saving
    The summary method in DebugController is now customizable, allowing it to wait longer before saving debug entries
  • Improved debugging methods
    Various methods in DebugController have been updated to better manage debug data and provide clearer exception messages

@codecov
Copy link

codecov bot commented Jun 18, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (6277877) 0.00% compared to head (27d8741) 0.00%.

Additional details and impacted files
@@           Coverage Diff            @@
##             master    #119   +/-   ##
========================================
  Coverage      0.00%   0.00%           
+ Complexity      231     230    -1     
========================================
  Files            21      21           
  Lines          1048    1046    -2     
========================================
+ Misses         1048    1046    -2     
Impacted Files Coverage Δ
src/Controller/DebugController.php 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@samdark
Copy link
Member

samdark commented Jun 20, 2023

What's the purpose of mutex in getting summary? What do you want to achieve?

@xepozz xepozz marked this pull request as draft July 9, 2023 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants