dawidchyrzynski / arduino-home-assistant

ArduinoHA allows to integrate an Arduino/ESP based device with Home Assistant using MQTT.
https://dawidchyrzynski.github.io/arduino-home-assistant/
GNU Affero General Public License v3.0
463 stars 112 forks source link

Missing checks if library is used in a wrong way? #230

Closed klausj1 closed 2 months ago

klausj1 commented 5 months ago

Hi,

Used library version: 2.0 Board: eps32C3 dev module

I had two issues which took me some time to find out:

I did not see an error message, it just did not work.

I am not sure if there are error messages already, because I did not manage to change the "Core Debug Level:" in the board configuration - I never saw any message, also not from other libraries.

If you are interested, I can also provide a more detailed description.

dawidchyrzynski commented 5 months ago

Hi @klausj1,

I will consider adding additional checks, possibly enabled by a macro; however, I am not fan of allocating resources (RAM/flash) for such scenarios. My general recommendation is to thoroughly review the documentation, as it describes your findings.

https://dawidchyrzynski.github.io/arduino-home-assistant/documents/library/introduction.html https://dawidchyrzynski.github.io/arduino-home-assistant/documents/library/compiler-macros.html

klausj1 commented 5 months ago

OK, Thx. Feel free to close this issue ;)