Closed elektron-bbs closed 2 years ago
Merging #1050 (d13ca74) into master (9376029) will increase coverage by
0.09%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #1050 +/- ##
==========================================
+ Coverage 61.33% 61.43% +0.09%
==========================================
Files 125 127 +2
Lines 9321 9371 +50
Branches 1475 1481 +6
==========================================
+ Hits 5717 5757 +40
- Misses 2534 2538 +4
- Partials 1070 1076 +6
Flag | Coverage Δ | |
---|---|---|
fhem | 52.70% <ø> (+0.21%) |
:arrow_up: |
modules | 61.43% <ø> (+0.09%) |
:arrow_up: |
perl | 91.61% <ø> (-0.14%) |
:arrow_down: |
unittests | 61.43% <ø> (+0.09%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
FHEM/14_SD_WS07.pm | 47.40% <ø> (ø) |
|
FHEM/lib/SD_ProtocolData.pm | 100.00% <ø> (ø) |
|
FHEM/lib/SD_Protocols.pm | 79.15% <0.00%> (-0.35%) |
:arrow_down: |
t/FHEM/14_SD_BELL/09_parseDatat.t | 80.00% <0.00%> (ø) |
|
t/FHEM/14_SD_UT/09_parseDatat.t | 80.00% <0.00%> (ø) |
|
FHEM/10_SD_Rojaflex.pm | 72.76% <0.00%> (+1.21%) |
:arrow_up: |
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 9376029...d13ca74. Read the comment docs.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[ ] Bugfix (please link issue)
[x] Feature enhancement
[ ] Documentation update
[ ] Unittest enhancement
[ ] other
What is the current behavior? (You can also link to an open issue here, if this describes the current behavior) Messages from the sensor of the Mebus HQ7312-2 weather station were not decoded. Please refer: https://github.com/RFD-FHEM/RFFHEM/issues/1024
What is the new behavior (if this is a feature change)? Messages are now decoded with the new protocol definition 7.1 from module 14_SD_WS07.pm.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: