Skip to content

Add xdebug #16

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

Merged
merged 2 commits into from
Oct 13, 2016
Merged

Add xdebug #16

merged 2 commits into from
Oct 13, 2016

Conversation

meron1122
Copy link
Contributor

No description provided.

@meron1122 meron1122 mentioned this pull request Oct 2, 2016
@maxpou
Copy link
Owner

maxpou commented Oct 5, 2016

Hello @meron1122,
Thank you for you Pull Request.
I didn't use xdebug. So, I need someone else to confirm this PR...

@@ -150,6 +150,10 @@ Run `docker-compose up -d` instead.

* Permission problem? See [this doc (Setting up Permission)](http://symfony.com/doc/current/book/installation.html#checking-symfony-application-configuration-and-setup)

* How to config Xdebug?
Xdebug is configured out of the box!
Just config your IDE to connect port `9001` and id key `PHPSTORM`
Copy link

@ghost ghost Oct 13, 2016

Choose a reason for hiding this comment

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

This needs more clarification.
I have switched the XDebug port in my PHPStorm config to 9001, but I can't seem to get any Debug connections going. Perhaps you'd like to elaborate a little bit on this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, just set here:
phpstorm-xdebug-config

Port 9001.

Copy link

Choose a reason for hiding this comment

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

Done that. Configured a server with path mappings etc.
Don't know what I'm missing. I'd love to give maxpou the feedback that this works so he can accept the PR, but I'm too dumb to get it working. :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link

Choose a reason for hiding this comment

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

I knew I was missing something vital. Thank you so much, it works perfectly! :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Finaly! :) Regards!.

@ghost
Copy link

ghost commented Oct 13, 2016

@maxpou With a little help from @meron1122, I can now confirm that his additions work perfectly. I suggest you accept his PR. 👍

@maxpou maxpou merged commit 6cc6f1c into maxpou:master Oct 13, 2016
@maxpou
Copy link
Owner

maxpou commented Oct 13, 2016

Good job and thank you guys :)

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.

2 participants