Skip to content

Fix more percritic in 00_SIGNALduino.pm#1148

Draft
sidey79 wants to merge 20 commits into
masterfrom
master_critic
Draft

Fix more percritic in 00_SIGNALduino.pm#1148
sidey79 wants to merge 20 commits into
masterfrom
master_critic

Conversation

@sidey79
Copy link
Copy Markdown
Contributor

@sidey79 sidey79 commented Jan 9, 2023

fileopen and sleep critic 5 fixed

  • Please check if the PR fulfills these requirements
  • Tests for the changes have been added / modified (needed for for bug fixes / features)
  • commandref has been added / updated (needed for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • Bugfix (please link issue)
  • 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)

  • What is the new behavior (if this is a feature change)?

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:

fileopen and sleep critic 5 fixed
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 9, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.49%. Comparing base (78b80d0) to head (295cb07).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1148      +/-   ##
==========================================
+ Coverage   88.92%   89.49%   +0.56%     
==========================================
  Files          43       43              
  Lines        2466     2466              
  Branches      170      170              
==========================================
+ Hits         2193     2207      +14     
+ Misses        119      105      -14     
  Partials      154      154              
Flag Coverage Δ
fhem 89.49% <ø> (+0.56%) ⬆️
modules 89.49% <ø> (+0.56%) ⬆️
perl 89.49% <ø> (+0.56%) ⬆️
unittests 89.49% <ø> (+0.56%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@HomeAutoUser
Copy link
Copy Markdown
Contributor

Sehe ich das richtig, dieser PR ist auch noch offen mit einem Review oder kommt hier noch mehr hinzu?

@sidey79
Copy link
Copy Markdown
Contributor Author

sidey79 commented Feb 13, 2023

Ja, da ich an den Fileopen Dingen was verändert habe, muss das noch in echt getestet werden.

@sidey79 sidey79 added the Perl::Critic code revisions label May 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Perl::Critic code revisions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants