Skip to content

Conversation

@chao2zhang
Copy link
Member

This is a follow-up of #3391 and should be merged as early as possible to ensure a smoother transition to the new Detekt 2.0 CLI #2692

@codecov
Copy link

codecov bot commented Jan 31, 2021

Codecov Report

Merging #3431 (da24da2) into master (56b5f77) will decrease coverage by 0.11%.
The diff coverage is 14.28%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3431      +/-   ##
============================================
- Coverage     80.37%   80.26%   -0.12%     
- Complexity     2777     2778       +1     
============================================
  Files           453      453              
  Lines          8377     8391      +14     
  Branches       1605     1605              
============================================
+ Hits           6733     6735       +2     
- Misses          775      787      +12     
  Partials        869      869              
Impacted Files Coverage Δ Complexity Δ
...tlab/arturbosch/detekt/DetektCreateBaselineTask.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...ab/arturbosch/detekt/extensions/DetektExtension.kt 52.38% <0.00%> (-2.62%) 7.00 <0.00> (ø)
.../io/gitlab/arturbosch/detekt/invoke/CliArgument.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
.../main/kotlin/io/gitlab/arturbosch/detekt/Detekt.kt 27.27% <16.66%> (-0.69%) 20.00 <1.00> (+1.00) ⬇️
...o/gitlab/arturbosch/detekt/internal/SharedTasks.kt 60.00% <50.00%> (-0.87%) 0.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 56b5f77...6cf77ce. Read the comment docs.

@schalkms schalkms merged commit fa79c99 into detekt:master Feb 1, 2021
@chao2zhang chao2zhang deleted the deprecate branch February 1, 2021 22:05
@cortinico cortinico added this to the 1.16.0 milestone Feb 19, 2021
This was referenced Mar 11, 2021
This was referenced Mar 11, 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.

4 participants