Support for Platformio - Githubissues

jnsbyr / esp8266-intexsbh20

MQTT WiFi remote control for the Intex PureSpa SB-H20 and SJB-HS whirlpools

Other

64 stars 20 forks source link

Support for Platformio #17

Closed fizcris closed 1 year ago

fizcris commented 2 years ago

This setup works with Platformio but maybe it would be best to create a separate branch for platform support.

.PIO created
Files were restructured
Some if loops fixed as they were exiting with a void return from bool functions

jnsbyr commented 2 years ago

Hey @fizcris, now we have 2 pull requests at the same time, that add PlatformIO support (see #16). I am still not sure which version I prefer. One thing that gives me a headache is that all pull requests change the formatting of the committed code. This makes it significantly more difficult to see where the real changes are. I will look into both solutions, but it will take I while.

jnsbyr commented 1 year ago

@fizcris + @piitaya

Found a way to add support for PlatformIO without breaking Arduino compatibility, using the esp8266 compiler settings for the D1 from fizcris fork (see d1c401f2d079b7de2c88f83b128eb1a91d0701ce).