Closed HomeAutoUser closed 6 months ago
Ich habe die Testdaten nicht gesehen. Hast Du ggf. vergessen diese einzuchecken?
Attention: 2 lines
in your changes are missing coverage. Please review.
Comparison is base (
fc62606
) 68.26% compared to head (d980db7
) 67.87%.
Files | Patch % | Lines |
---|---|---|
FHEM/14_SD_UT.pm | 75.00% | 0 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Ich habe die Testdaten nicht gesehen. Hast Du ggf. vergessen diese einzuchecken?
Lange war der letzte PR her lach und daher musste ich mich erstmal wieder hineinfuchsen. Testdaten wurden hinterlegt und du solltest sie nun auch sehen können :-) smile
@HomeAutoUser Hat sich zwar soeben erledigt, aber siehst du eigentlich jetzt mein review weiter oben?
@HomeAutoUser Hat sich zwar soeben erledigt, aber siehst du eigentlich jetzt mein review weiter oben?
Ich habe soeben noch einfach nachgesehen hier und habe nichts gefunden :(
Eigenartig, naja, ich habe die Änderungen jetzt bestätigt. Bleibt nur noch:
codecov/project — 59.09% (-1.14%) compared to fc62606
Warum das noch mäkelt kann wahrscheinlich wieder mal nur @sidey79 beantworten :-(
Eigenartig, naja, ich habe die Änderungen jetzt bestätigt. Bleibt nur noch:
codecov/project — 59.09% (-1.14%) compared to fc62606
Warum das noch mäkelt kann wahrscheinlich wieder mal nur @sidey79 beantworten :-(
Bin ein wenig ratlos, es gibt Änderungen in der Testabdeckung an Stellen, an denen nichts verändert wurde.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[ ] Bugfix (please link issue)
[x] Feature enhancement
[ ] Documentation update
[x] Unittest enhancement
[ ] other
What is the current behavior? (You can also link to an open issue here, if this describes the current behavior) -- nothing to decode
What is the new behavior (if this is a feature change)? -- LED actor can control via FHEM https://github.com/RFD-FHEM/RFFHEM/issues/1207
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information: