Closed elektron-bbs closed 10 months ago
Merging #1184 (96a2b5c) into master (4586841) will increase coverage by
0.62%
. The diff coverage is73.07%
.
@@ Coverage Diff @@
## master #1184 +/- ##
==========================================
+ Coverage 67.12% 67.74% +0.62%
==========================================
Files 133 135 +2
Lines 9894 9936 +42
Branches 1586 1591 +5
==========================================
+ Hits 6641 6731 +90
+ Misses 1944 1897 -47
+ Partials 1309 1308 -1
Flag | Coverage Δ | |
---|---|---|
fhem | 57.54% <73.07%> (+0.78%) |
:arrow_up: |
modules | 67.74% <73.07%> (+0.62%) |
:arrow_up: |
perl | 89.63% <ø> (ø) |
|
unittests | 67.74% <73.07%> (+0.62%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Files Changed | Coverage Δ | |
---|---|---|
FHEM/lib/SD_ProtocolData.pm | 100.00% <ø> (ø) |
|
t/FHEM/14_SD_UT/03_set.t | 92.30% <ø> (ø) |
|
FHEM/14_SD_UT.pm | 49.94% <72.00%> (+1.18%) |
:arrow_up: |
FHEM/00_SIGNALduino.pm | 63.96% <100.00%> (-0.35%) |
:arrow_down: |
... and 8 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
@elektron-bbs, was denkst du, ist es auch an der Zeit, die SD_UT im SVN zu aktualisieren?
Das macht sicher erst richtig Sinn, wenn die 00_SIGNALduino.pm dort aktualisiert wurde, da ja die %matchListSIGNALduino sonst nicht passt.
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) No decoding of the radio messages of two remote controls for ceiling fans, see https://forum.fhem.de/index.php?topic=134121.0 and https://forum.fhem.de/index.php?msg=1281573.
What is the new behavior (if this is a feature change)? Reception and transmission of radio messages enabled.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: