Skip to content

Simplify code in RedundantSuspendModifier rule#3580

Merged
schalkms merged 1 commit into
detekt:masterfrom
schalkms:simplify-RedundantSuspendModifier
Mar 20, 2021
Merged

Simplify code in RedundantSuspendModifier rule#3580
schalkms merged 1 commit into
detekt:masterfrom
schalkms:simplify-RedundantSuspendModifier

Conversation

@schalkms

Copy link
Copy Markdown
Member

No description provided.

@picklebento picklebento added this to the 1.17.0 milestone Mar 19, 2021
@codecov

codecov Bot commented Mar 19, 2021

Copy link
Copy Markdown

Codecov Report

Merging #3580 (79b0aec) into master (1b45d78) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

❗ Current head 79b0aec differs from pull request most recent head 27c8fd1. Consider uploading reports for the commit 27c8fd1 to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3580      +/-   ##
============================================
- Coverage     77.52%   77.51%   -0.01%     
  Complexity     2829     2829              
============================================
  Files           464      464              
  Lines          8769     8767       -2     
  Branches       1713     1713              
============================================
- Hits           6798     6796       -2     
  Misses         1047     1047              
  Partials        924      924              
Impacted Files Coverage Δ Complexity Δ
...etekt/rules/coroutines/RedundantSuspendModifier.kt 57.14% <50.00%> (-1.95%) 20.00 <5.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 1b45d78...27c8fd1. Read the comment docs.

@schalkms schalkms merged commit c32b041 into detekt:master Mar 20, 2021
@schalkms schalkms deleted the simplify-RedundantSuspendModifier branch March 20, 2021 09:57
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.

3 participants