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

Added debug mode to enable use of breakpoints when debugging #37

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

KThaulow
Copy link
Contributor

@KThaulow KThaulow commented Apr 27, 2024

Added debug mode to the launch.json configuration and moved configuration.yaml to be used by the homeassistant requirement. Modified develop script to be able to be used by the existing run task.

This integration can now be run by pressing F5 and running the Home Assistant: Debug configuration.

This partially solves the issue raised in #36 as debugging is now supported with breakpoints. However, it did not solve the issue of using breakpoints when attaching the debugger after launching the home assistant task.

Added debug mode to launch configuration and moved configruation.yaml to be used by the homeassistant requirement. Modified develop script to be able to be used by the existing run task.
@toringer
Copy link
Owner

"Run Task" and debug now have their "config" folder in different locations. This results in different db and config depending on how HA is started. Was this done intentionally? I would prefer to have a common location.

…nd updated develop script and documentatation
@KThaulow
Copy link
Contributor Author

"Run Task" and debug now have their "config" folder in different locations. This results in different db and config depending on how HA is started. Was this done intentionally? I would prefer to have a common location.

Good catch, that was not intentional. I've updated it to use the shared location in the config/ directory, as that seems to be the standard here and here for where to put the configuration.yaml. Let me know if you prefer to keep it under the .devcontainer/hass instead.

@toringer toringer merged commit 76b4594 into toringer:master Apr 29, 2024
3 checks passed
@toringer
Copy link
Owner

config/ is just fine. Thank you for the contribution 👍

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