search

bpaauwe / WeatherFlow

UDI Polyglot based node server for the WeatherFlow smart weather station
Other
4 stars 2 forks source link

Added heartbeat #3

Closed jimboca closed 6 years ago

jimboca commented 6 years ago

Let me know if you have any issue with this. Looks like my editor removed unneeded spaces from the end of some lines..

jimboca commented 6 years ago

@bpaauwe Let me know if you like these changes are would prefer something different?

bpaauwe commented 6 years ago

Thanks Jim.

I believe I have everything merged. I cleaned up and reorganized them a bit as I merged them. The main changes are that I split out the version number change and removed all the white space changes. I'd rather have a separate code cleanup patch than have it embedded with a new feature. I also added a bit of text to the commit messages so make sure I didn't say something wrong there.

For future reference, you don't need to worry to much about conflicts, I think I can resolve those pretty easy when I merge. If not, I'd let you know.

jimboca commented 6 years ago

Looks good, thanks!