Skip to content
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

Use list config for DataClassContainsFunctions>conversionFunctionPrefix #5119

Merged
merged 1 commit into from Jul 30, 2022

Conversation

BraisGabin
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Jul 23, 2022

Codecov Report

Merging #5119 (a48355c) into main (855065b) will decrease coverage by 0.00%.
The diff coverage is 90.00%.

@@             Coverage Diff              @@
##               main    #5119      +/-   ##
============================================
- Coverage     84.91%   84.91%   -0.01%     
- Complexity     3614     3615       +1     
============================================
  Files           502      502              
  Lines         11887    11889       +2     
  Branches       2237     2238       +1     
============================================
+ Hits          10094    10095       +1     
  Misses          690      690              
- Partials       1103     1104       +1     
Impacted Files Coverage Δ
...h/detekt/rules/style/DataClassContainsFunctions.kt 92.00% <90.00%> (-3.66%) ⬇️

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 855065b...a48355c. Read the comment docs.

@BraisGabin BraisGabin force-pushed the use-config-list-DataClassContainsFunctions branch from 4a9826f to eafa042 Compare July 23, 2022 14:09
@BraisGabin BraisGabin merged commit fc23515 into main Jul 30, 2022
21 checks passed
@BraisGabin BraisGabin deleted the use-config-list-DataClassContainsFunctions branch July 30, 2022 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants