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.

Sign up for GitHub

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

Jump to bottom

chore(docker): add xdebug support #81

Merged
merged 25 commits into from
Mar 26, 2024
Merged

chore(docker): add xdebug support #81

merged 25 commits into from
Mar 26, 2024

Conversation

jokesterfr
Copy link
Contributor

@jokesterfr jokesterfr commented Feb 28, 2024 •
edited
Loading

Questions Answers
Description? XDEBUG is a general purpose tool for debugging a PHP application. PrestaShop can be targeted with this useful tool, even from a docker container environment, and flashlight be used as a full featured debugging environment. See the following link to reach the PHP XDEBUG compatibility matrix: https://xdebug.org/docs/compat.
Bonus: fix the extra --wait call in the hydrate script to killall mysqld instances (was waiting infinitely).
Type? new feature
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #74, #83
Sponsor company PrestaShop SA
How to test? Please see the provided example.

@jokesterfr jokesterfr self-assigned this Feb 28, 2024
@jokesterfr jokesterfr added the wip label Feb 28, 2024
@jokesterfr jokesterfr requested review from M0rgan01 and Venatum March 11, 2024 08:53
@jokesterfr jokesterfr marked this pull request as ready for review March 11, 2024 08:53
Venatum
Venatum approved these changes Mar 11, 2024
View reviewed changes
examples/xdebug-prestashop/docker-compose.yml Outdated Show resolved Hide resolved
@jokesterfr jokesterfr added ready to review Looking for a pair review and removed wip labels Mar 12, 2024
@jokesterfr jokesterfr merged commit 060ad80 into main Mar 26, 2024
3 checks passed
@jokesterfr jokesterfr deleted the fea/add-xdebug-support branch March 26, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Venatum Venatum Venatum approved these changes

@M0rgan01 M0rgan01 M0rgan01 approved these changes

Assignees

@jokesterfr jokesterfr

Labels
ready to review Looking for a pair review
Projects
PR Dashboard
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add XDEBUG support
3 participants
@jokesterfr @Venatum @M0rgan01