Skip to content

Conversation

@chao2zhang
Copy link
Member

@chao2zhang chao2zhang commented Feb 19, 2022

  • Move our own Gradle scripts to the new reporting configuration API.
  • Use Constants defined in DetektExtension

@codecov
Copy link

codecov bot commented Feb 19, 2022

Codecov Report

Merging #4589 (e5385b7) into main (f15ce50) will increase coverage by 84.49%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #4589       +/-   ##
===========================================
+ Coverage        0   84.49%   +84.49%     
- Complexity      0     3335     +3335     
===========================================
  Files           0      481      +481     
  Lines           0    11147    +11147     
  Branches        0     2040     +2040     
===========================================
+ Hits            0     9419     +9419     
- Misses          0      697      +697     
- Partials        0     1031     +1031     
Impacted Files Coverage Δ
...itlab/arturbosch/detekt/api/FileProcessListener.kt 100.00% <0.00%> (ø)
...ab/arturbosch/detekt/rules/bugs/ExitOutsideMain.kt 71.42% <0.00%> (ø)
...ab/arturbosch/detekt/cli/runners/ConfigExporter.kt 88.88% <0.00%> (ø)
...tekt/rules/exceptions/NotImplementedDeclaration.kt 75.00% <0.00%> (ø)
.../io/gitlab/arturbosch/detekt/rules/GuardClauses.kt 20.00% <0.00%> (ø)
.../arturbosch/detekt/rules/style/UseIsNullOrEmpty.kt 55.84% <0.00%> (ø)
.../kotlin/io/gitlab/arturbosch/detekt/cli/CliArgs.kt 100.00% <0.00%> (ø)
...bosch/detekt/rules/exceptions/ReturnFromFinally.kt 85.71% <0.00%> (ø)
...io/github/detekt/tooling/internal/PluginsHolder.kt 100.00% <0.00%> (ø)
...detekt/rules/exceptions/ThrowingExceptionInMain.kt 90.90% <0.00%> (ø)
... and 471 more

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 f15ce50...e5385b7. Read the comment docs.

@3flex
Copy link
Member

3flex commented Feb 20, 2022

Concatenate all include statements into one.

Can we not do this please? It makes it a bit more annoying to tweak the settings script when using Gradle's composite build feature, where you need to change include directives to includeBuild and back for example.

Unless there's some advantage to doing this I'm not aware of?

@chao2zhang
Copy link
Member Author

The only advantage is the visual appearance - I have reverted the change.

@3flex 3flex merged commit 6c3c28d into detekt:main Feb 21, 2022
@chao2zhang chao2zhang deleted the gradleclean branch February 21, 2022 17:08
@cortinico cortinico added this to the 1.20.0 milestone Feb 22, 2022
@cortinico cortinico added the housekeeping Marker for housekeeping tasks and refactorings label Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

housekeeping Marker for housekeeping tasks and refactorings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants