Closed Ernst79 closed 1 year ago
Base: 74.60% // Head: 75.11% // Increases project coverage by +0.50%
:tada:
Coverage data is based on head (
e7490b4
) compared to base (27e5982
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Looks good! Running tests on my side worked. When doing that I also added (locally) a test case with data collected from one of our sensors.
A suggestion - if this is an isolated case - rather than changing the whole flake8 max-line-length
setting to 106 just because of this line, you could use an inline comment for flake8 to ignore the error in that specific line: # noqa: E501
.
Cheers!
Thanks for checking. Fixed your comment, that is indeed a better solution. flake8 and black where changing each other back and forth.
feat: add possibility to include mac in payload (fix for #42)