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

Create xdebug addon #89

Merged
merged 3 commits into from
Dec 9, 2022
Merged

Create xdebug addon #89

merged 3 commits into from
Dec 9, 2022

Conversation

froboy
Copy link
Contributor

@froboy froboy commented Aug 30, 2022

A very simple addon to turn xdebug on or off (that mirrors DDEV's function)

shelane
shelane previously approved these changes Aug 30, 2022
Copy link
Member

@shelane shelane left a comment

Choose a reason for hiding this comment

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

Looks very similar to a custom command that I have.

@froboy
Copy link
Contributor Author

froboy commented Aug 30, 2022

@shelane thanks. Yea... it's been hanging out for a while and I figure it'd be nice to share. I could set no option to toggle the state, but figure doing nothing is always... non-destructive.

@shelane shelane dismissed their stale review October 14, 2022 14:08

Needs readme update

@shelane
Copy link
Member

shelane commented Oct 14, 2022

Please create in your PR an update to the readme file with this included in alphabetical order in the list.

@shelane shelane self-requested a review October 14, 2022 14:09
Copy link
Member

@shelane shelane left a comment

Choose a reason for hiding this comment

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

Readme needs update

@froboy froboy requested review from shelane and removed request for sean-e-dietrich October 14, 2022 14:28
@shelane shelane self-requested a review October 14, 2022 16:55
Copy link
Member

@shelane shelane left a comment

Choose a reason for hiding this comment

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

See inline comment

xdebug/xdebug Outdated
fin up
;;
*)
echo "Please tell me 'on' or 'off'."
Copy link
Member

Choose a reason for hiding this comment

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

Sorry, just one more nit-picky little thing. For consistency of verbiage with other commands, can you change this to:

"Usage: fin xdebug <on|off>"

and add exit 1 on the next line?

@achekulaev achekulaev merged commit 1b0e80e into docksal:master Dec 9, 2022
@froboy froboy deleted the patch-1 branch February 24, 2025 22:27
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