search

zigpy / zigpy-znp

TI CC2531, CC13x2, CC26x2 radio support for Zigpy and ZHA
GNU General Public License v3.0
146 stars 40 forks source link

Add mypy and more flake checks #138

Closed mdeweerd closed 2 years ago

puddly commented 2 years ago

Thank you!

I've submitted a PR to your fork (https://github.com/mdeweerd/zigpy-znp/pull/1) that fixes the remaining issues preventing this from being merged.

codecov-commenter commented 2 years ago

Codecov Report

Merging #138 (fbc9fe8) into dev (515f4a1) will decrease coverage by 0.07%. The diff coverage is 94.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #138      +/-   ##
==========================================
- Coverage   98.66%   98.59%   -0.08%     
==========================================
  Files          44       44              
  Lines        3907     3918      +11     
==========================================
+ Hits         3855     3863       +8     
- Misses         52       55       +3
Impacted Files Coverage Δ
zigpy_znp/api.py 96.81% <84.21%> (-0.74%) :arrow_down:
zigpy_znp/tools/common.py 100.00% <100.00%> (ø)
zigpy_znp/types/basic.py 100.00% <100.00%> (ø)
zigpy_znp/types/commands.py 100.00% <100.00%> (ø)
zigpy_znp/types/cstruct.py 100.00% <100.00%> (ø)
zigpy_znp/types/named.py 100.00% <100.00%> (ø)
zigpy_znp/utils.py 100.00% <100.00%> (ø)
zigpy_znp/zigbee/application.py 95.85% <100.00%> (ø)
zigpy_znp/znp/security.py 95.18% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 515f4a1...fbc9fe8. Read the comment docs.