Skip to content

Disable xdebug warning #126

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Disable xdebug warning #126

wants to merge 1 commit into from

Conversation

xepozz
Copy link
Member

@xepozz xepozz commented Jul 18, 2024

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

I'm not sure, but now errors aren't showed

@xepozz xepozz requested a review from a team July 18, 2024 09:14
@xepozz xepozz added the status:code review The pull request needs review. label Jul 18, 2024
@@ -149,6 +149,6 @@ private function isXdebugStackAvailable(): bool
}

// Xdebug 3 and later, proper mode is required
return str_contains(ini_get('xdebug.mode'), 'develop');
return str_contains(ini_get('xdebug.mode'), 'debug');
Copy link
Member

Choose a reason for hiding this comment

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

mode docs:

develop Enables Development Helpers...

xdebug_get_function_stack() is included in development helpers.


What warning are you showing?

@xepozz
Copy link
Member Author

xepozz commented Sep 8, 2024

Master branch:
image

PR's branch:
image

Also terminal:
image

@vjik
Copy link
Member

vjik commented Sep 10, 2024

Check you PHP settings. I tested with xdebug.mode=develop and it's work.

@xepozz
Copy link
Member Author

xepozz commented Sep 19, 2024

Check you PHP settings. I tested with xdebug.mode=develop and it's work.

❯ php -i | grep xdebug.mode
             Enabled Features (through 'xdebug.mode' setting)             
xdebug.mode => develop => develop

It works, but with silent warnings

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