Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove instances of double mutability #5899

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.

Sign up for GitHub

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

Jump to bottom
Merged
schalkms merged 2 commits into from
Mar 13, 2023
Merged

Remove instances of double mutability #5899

schalkms merged 2 commits into from
Mar 13, 2023

Conversation

@3flex
Copy link
Member

@3flex 3flex commented Mar 13, 2023

No description provided.

@github-actions github-actions bot added the rules label Mar 13, 2023
@github-actions
Copy link

github-actions bot commented Mar 13, 2023 •
edited
Loading

Warnings
⚠️ It looks like this PR contains functional changes without a corresponding test.

Generated by 🚫 dangerJS against c7ff8b1

@3flex 3flex added housekeeping Marker for housekeeping tasks and refactorings rules and removed rules labels Mar 13, 2023
@3flex 3flex marked this pull request as ready for review March 13, 2023 10:48
@codecov
Copy link

codecov bot commented Mar 13, 2023 •
edited
Loading

Codecov Report

Merging #5899 (c7ff8b1) into main (f02e12f) will decrease coverage by 0.03%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##               main    #5899      +/-   ##
============================================
- Coverage     84.59%   84.56%   -0.03%     
+ Complexity     3784     3782       -2     
============================================
  Files           546      546              
  Lines         12943    12933      -10     
  Branches       2273     2271       -2     
============================================
- Hits          10949    10937      -12     
- Misses          861      863       +2     
  Partials       1133     1133
Impacted Files Coverage Δ
...github/detekt/tooling/dsl/ExtensionsSpecBuilder.kt 87.50% <0.00%> (-7.96%) ⬇️
...b/arturbosch/detekt/rules/complexity/LargeClass.kt 93.93% <100.00%> (-0.94%) ⬇️
...b/arturbosch/detekt/rules/complexity/LongMethod.kt 91.11% <100.00%> (-1.05%) ⬇️
...tlab/arturbosch/detekt/rules/bugs/LateinitUsage.kt 92.59% <100.00%> (ø)
...rbosch/detekt/rules/performance/CouldBeSequence.kt 87.50% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@3flex 3flex force-pushed the double-mutability branch from 3e90b75 to c7ff8b1 Compare March 13, 2023 11:28
@schalkms schalkms added this to the 1.23.0 milestone Mar 13, 2023
@schalkms schalkms merged commit 0123e29 into detekt:main Mar 13, 2023
@3flex 3flex deleted the double-mutability branch March 13, 2023 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schalkms schalkms schalkms approved these changes

Assignees

No one assigned

Labels

housekeeping Marker for housekeeping tasks and refactorings rules

Projects

None yet

Milestone

1.23.0

Development

Successfully merging this pull request may close these issues.

2 participants

@3flex @schalkms