-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upstreaming it? #17
Comments
From my experience, it creates a lot of extra effort to merge a component into esphome (or HomeAssistant) main branch and then maintain it there. Therefore I don't have plans to do this at the moment. |
Would be nice to at least get feedback and then decide if it's worth the time right now. |
https://esphome.io/components/sml.html is offered upstream, does this differ significantly? |
SML and D0 are 2 different communication protocols, but both use OBIS codes. |
ah ok, so not synonym and still needed to become mainlined. |
Have you considered making this part of ESPHome itself and create a pull request like the Optolink Team does in esphome/esphome#4453 ?
The text was updated successfully, but these errors were encountered: