Skip to content

Enable ParameterListWrapping rule on detekt codebase #4178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

3flex
Copy link
Member

@3flex 3flex commented Oct 12, 2021

This is a standard ktlint rule.

@3flex 3flex added the housekeeping Marker for housekeeping tasks and refactorings label Oct 12, 2021
@3flex 3flex added this to the 1.19.0 milestone Oct 12, 2021
@codecov
Copy link

codecov bot commented Oct 12, 2021

Codecov Report

Merging #4178 (eaf891f) into main (954bd5e) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #4178      +/-   ##
============================================
- Coverage     83.45%   83.44%   -0.02%     
- Complexity     3181     3190       +9     
============================================
  Files           465      465              
  Lines          9104     9119      +15     
  Branches       1774     1779       +5     
============================================
+ Hits           7598     7609      +11     
  Misses          572      572              
- Partials        934      938       +4     
Impacted Files Coverage Δ
...ab/arturbosch/detekt/rules/style/WildcardImport.kt 90.00% <0.00%> (-4.12%) ⬇️
...urbosch/detekt/rules/performance/ArrayPrimitive.kt 71.42% <0.00%> (-3.58%) ⬇️
...rbosch/detekt/rules/complexity/NestedBlockDepth.kt 91.83% <0.00%> (+0.72%) ⬆️

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 954bd5e...eaf891f. Read the comment docs.

@3flex 3flex enabled auto-merge (squash) October 12, 2021 11:43
@3flex 3flex merged commit 5904d25 into detekt:main Oct 12, 2021
@3flex 3flex deleted the enable-parameterwrapping-rule branch October 12, 2021 20:12
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.

2 participants