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

Add remote debugging by default in dev docker-compose #541

Merged

Conversation

leonkenneth
Copy link
Contributor

Pas sûr que ce soit utile, vu que je viens de setup mon environnement je peux avoir raté quelque chose, mais j'ai l'impression que ça m'étais nécessaire pour remote-debug avec VSCode

Copy link
Collaborator

@fchabouis fchabouis left a comment

Choose a reason for hiding this comment

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

tu t'en sers comment ?

@leonkenneth
Copy link
Contributor Author

Avec la configuration "Python Remote Debugger" by default

Capture d’écran 2025-02-03 à 13 31 47

@fchabouis
Copy link
Collaborator

J'ai testé la PR chez moi en local, je ne peux plus acceder au site à l'adresse http://localhost:8000/.
Ça fonctionne chez toi de lancer le docker compose up -det de pouvoir tester l'API sur http://localhost:8000/ ?

@leonkenneth leonkenneth force-pushed the add-remote-debugging-by-default-in-dev-dockercompose branch from ae15bd1 to 4ab9b5c Compare February 3, 2025 16:09
@leonkenneth
Copy link
Contributor Author

Oui, mais je viens de comprendre que l'option --wait-for-client attend qu'un debugger s'attache avant de lancer (comme son nom l'indique).

Ca permet de debug le startup j'imagine, mais c'est vrai que ça force à lancer un debugger. Je viens de la retirer, tu as plus de succès ?

@fchabouis
Copy link
Collaborator

J'ai testé chez moi sur VS Code et effectivement j'ai pu lancer un remote debug attaché au serveur web qui tourne dans le container docker. Donc avec mon setup, ce changement ne casse rien. Reste à voir avec les conf des autres dev de l'équipe si c'est également le cas.

@leonkenneth leonkenneth enabled auto-merge February 6, 2025 12:54
@leonkenneth leonkenneth merged commit 39d3325 into main Feb 6, 2025
5 checks passed
@leonkenneth leonkenneth deleted the add-remote-debugging-by-default-in-dev-dockercompose branch February 6, 2025 13:07
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