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

Add example for profiler.enable with xdebug profiling enabled #58

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

glensc
Copy link
Contributor

@glensc glensc commented Dec 19, 2020

This adds an example to Detect XDebug triggers and skip profiling:

See perftools/xhgui#104 (comment)

@glensc
Copy link
Contributor Author

glensc commented Dec 19, 2020

This needs to be validated first as xdebug_is_enabled() description in phpstorm stubs says "Return whether stack traces would be shown in case of an error or not"

@glensc
Copy link
Contributor Author

glensc commented Dec 19, 2020

Perhaps xdebug_info gives he accurate info:

➔ php -r "xdebug_info();"|grep -i xdebug.mode
xdebug.mode => develop => develop

which is just in value:

$ php -r 'var_dump(ini_get("xdebug.remote_autostart"));'
string(1) "0"

@glensc glensc marked this pull request as draft January 3, 2021 21:28
README.md Outdated Show resolved Hide resolved
@glensc glensc changed the base branch from master to main January 18, 2022 19:52
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.

1 participant