Marc Hörsken
Marc Hörsken
> [When adding new integrations, limit included platforms to a single platform. Please reduce this PR to a single platform](https://developers.home-assistant.io/docs/review-process/#home-assistant-core) Is this a must or a should? I am asking...
> > [When adding new integrations, limit included platforms to a single platform. Please reduce this PR to a single platform](https://developers.home-assistant.io/docs/review-process/#home-assistant-core) > > Is this a must or a should?...
I will follow up with cover and light in separate PRs since I have it ready locally, but scenes would allow at least versatile indirect control already.
Ok, will do the change then. Adding more than one platform is not an option right? Because I have lights, awnings and scenes fully working. Rotating PR content and documentation...
Switched the PR from scenes to awnings as covers now. > Yes, but handling a state in your integration is probably the most interesting part which also defines how your...
The PR has reached 100% test coverage now 🎉
I must say: This is starting to become a little bit frustrating, it feels like chasing a moving target...
I am just trying to understand the reasoning behind your comments, especially since I tried to follow the developer documentation and examples available in the codebase. I am going to...
Okay, I tried to incorporate more feedback. Can you please elaborate which of the parts regarding a) different mocking, b) parameterized tests and c) snapshot tests would be important/required to...
Yes, I am on Discord with the same username, just pinged you there. Have a good night, too. Question to consider regarding the mocking for tomorrow: would it be okay...