Skip to content

Fix rule 0912 to include type of var in rule message#597

Merged
mnojek merged 4 commits intomasterfrom
fix/592
Apr 11, 2022
Merged

Fix rule 0912 to include type of var in rule message#597
mnojek merged 4 commits intomasterfrom
fix/592

Conversation

@mnojek
Copy link
Member

@mnojek mnojek commented Apr 9, 2022

Fixes #592

@codecov-commenter
Copy link

codecov-commenter commented Apr 9, 2022

Codecov Report

Merging #597 (db7b196) into master (5b5f324) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #597   +/-   ##
=======================================
  Coverage   97.34%   97.34%           
=======================================
  Files          23       23           
  Lines        2640     2640           
=======================================
  Hits         2570     2570           
  Misses         70       70           
Impacted Files Coverage Δ
robocop/checkers/misc.py 99.13% <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 5b5f324...db7b196. Read the comment docs.

@mnojek mnojek changed the title Fix/592 Fix rule 0912 to include type of var in rule message Apr 10, 2022
@mnojek mnojek merged commit 0236cf1 into master Apr 11, 2022
@mnojek mnojek deleted the fix/592 branch April 11, 2022 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Rule 0912 Incorrectly recommend to use ${EMPTY} for List and Dictionaries

3 participants