Skip to content

UnnecessaryLet: report when implicit parameter isn't used#3794

Merged
schalkms merged 1 commit intodetekt:mainfrom
t-kameyama:issue_3786
May 18, 2021
Merged

UnnecessaryLet: report when implicit parameter isn't used#3794
schalkms merged 1 commit intodetekt:mainfrom
t-kameyama:issue_3786

Conversation

@t-kameyama
Copy link
Copy Markdown
Contributor

Fixes #3786

@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2021

Codecov Report

Merging #3794 (77fc8fb) into main (2fbac45) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #3794   +/-   ##
=========================================
  Coverage     83.51%   83.51%           
  Complexity     2914     2914           
=========================================
  Files           452      452           
  Lines          8758     8760    +2     
  Branches       1665     1665           
=========================================
+ Hits           7314     7316    +2     
  Misses          542      542           
  Partials        902      902           
Impacted Files Coverage Δ Complexity Δ
...ab/arturbosch/detekt/rules/style/UnnecessaryLet.kt 82.35% <100.00%> (+0.72%) 13.00 <0.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 2fbac45...77fc8fb. Read the comment docs.

@schalkms schalkms added this to the 1.18.0 milestone May 18, 2021
@schalkms schalkms merged commit e5bc819 into detekt:main May 18, 2021
@t-kameyama t-kameyama deleted the issue_3786 branch May 18, 2021 21:44
@cortinico cortinico mentioned this pull request May 20, 2021
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.

UnnecessaryLet false negative

3 participants