-
Notifications
You must be signed in to change notification settings - Fork 404
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
Simplify interface and add type hints #2608
Conversation
1714748
to
b09543d
Compare
This module shouldn't be necessary on the long run. Remove pieces from it whenever possible.
Enforce a total coverage of 40% of the code. Related pwr-Solaar#1097
Create coverage.xml, upload it to GitHub CI and visualize with codecov. Setup instruction: - Install codecov for project https://github.com/settings/installations/55029514 - Add CODECOV_TOKEN in the GitHub CI project secrets Related pwr-Solaar#1097
Introduce unit tests. Related pwr-Solaar#2273
Introduce unit tests. Related pwr-Solaar#2273
- Remove comments with unused receivers - Simplify receiver hardcoded info
Add test for it.
b09543d
to
3651d83
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #2608 +/- ##
=========================================
Coverage ? 47.92%
=========================================
Files ? 52
Lines ? 14337
Branches ? 2817
=========================================
Hits ? 6871
Misses ? 7080
Partials ? 386
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
No description provided.