-
Notifications
You must be signed in to change notification settings - Fork 2
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
Valet restart #1
Comments
Hi, I'm glad you like xDebugToggler. This is a kind of a known issue. Sometimes even running For the next version; running commands with Thanks for noticing. |
A restart valet option would be nice. |
Hello, I've just discovered this wonderful little tool and this exact issue is cropping up! Valet requires that the php services is run by root and not the user, so when this tool tries to restart the brew service it doesn't use Really hoping at some stage this might be incorporated! Thank you very much @deligoez |
https://developer.apple.com/forums/thread/91350 Of course, having looked into this a little, it appears nothing will be easy! 🙄 |
Hi Everybody, Thanks for your suggestions. Before diving into running
I'll start to implement this today. |
Is this project maintained anymore? Really would like it to include valet restart. In the meantime you can just add this to your ~/.bashrc or ~/.zshrc file. Then you'll be able to turn on/off xdebug from the terminal
Then re-source using |
Thanks for this really nifty little tool! I found it through the Laravel newsletter.
I have a bit of an issue. I use Laravel Valet for local development. I have Xdebug installed (the ext-xdebug.ini-file inside conf.d, etc) and it works fine. However, if I disable xdebug using the toggler and then re-enable it, xdebug appears not to be enabled. Even checking "--all" for which services to restart, xdebug is not active again until I run
valet restart
. I don't know why this is the case.One possible solution to this, would be adding an option for a "custom command" to run after toggling (instead of the brew services commands). Just a thought.
Thanks again.
The text was updated successfully, but these errors were encountered: