Hello! Again, thanks for the great work on this. I'd like to be able to help make it more awesome, if you'd be interested in that!
This first pull request contains some very minor suggestions that have helped me so far when setting up my development environment:
The Energenie name in the HA discovery packets was lower case, which made it feel lesser alongside the other devices in my HA instance that have capitalised brand names 😅
There was a missing semi-colon; tiny
This one might be more controversial: Could I suggest we bring through the package version into the app runtime, so that it can be a) used at logging start but also b) in the discovery packages; the firmware property that the sw discovery property ends up in, is perfectly suited for displaying version numbers
Hope you like this and are interested in further collaboration. I've got another PR I'll submit shortly that might be even further controversial, so I'll split it into its own for a separate discussion
Accidentally submitted the PR against my develop branch, rather than a specific one for these explicit features and didn't know how to rename/repoint the PR because I'm new to PR's of this nature 😅
Hello! Again, thanks for the great work on this. I'd like to be able to help make it more awesome, if you'd be interested in that!
This first pull request contains some very minor suggestions that have helped me so far when setting up my development environment:
firmware
property that thesw
discovery property ends up in, is perfectly suited for displaying version numbersHope you like this and are interested in further collaboration. I've got another PR I'll submit shortly that might be even further controversial, so I'll split it into its own for a separate discussion