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

Update NoEmptyClassBody.kt #7033

Merged
merged 1 commit into from Mar 9, 2024
Merged

Update NoEmptyClassBody.kt #7033

merged 1 commit into from Mar 9, 2024

Conversation

tfcporciuncula
Copy link
Contributor

Same thing I did here a while ago: NoEmptyClassBody overlaps with EmptyClassBlock and the current docs don't mention it.

Updating the KDoc this time so it's fixed permanently in future versions (thanks @3flex!).

@detekt-ci
Copy link
Collaborator

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

Generated by 🚫 dangerJS against c0a57e1

Copy link

codecov bot commented Mar 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.79%. Comparing base (312831b) to head (c0a57e1).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #7033   +/-   ##
=========================================
  Coverage     83.79%   83.79%           
  Complexity     3941     3941           
=========================================
  Files           579      579           
  Lines         12119    12119           
  Branches       2511     2511           
=========================================
  Hits          10155    10155           
  Misses          710      710           
  Partials       1254     1254           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@schalkms schalkms added this to the 2.0.0 milestone Mar 9, 2024
@schalkms schalkms merged commit 8e952c8 into detekt:main Mar 9, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants