General Post POC refactoring.

dalinicus / homeassistant-acinfinity

AC Infinity integration for Home Assistant for UIS based controllers

MIT License

75 stars 4 forks source link

General Post POC refactoring. #9

Closed dalinicus closed 1 year ago

dalinicus commented 1 year ago

Add unit test coverage
Add linting and static code analysis
Normalize and simplify entity ids
Split entities into correct platforms (ie. binary_sensor and number)
Simplify _data updating process and property fetching, keeping whole returned data structure

Resolves #8

github-advanced-security[bot] commented 1 year ago

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

codecov-commenter commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@ea370ba). Click here to learn what that means. Patch has no changes to coverable lines.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #9 +/- ## ======================================= Coverage ? 99.62% ======================================= Files ? 9 Lines ? 269 Branches ? 0 ======================================= Hits ? 268 Misses ? 1 Partials ? 0 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.