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

Fix ComplexMethod debt and refactor code #5029

Merged
merged 2 commits into from Jul 6, 2022
Merged

Fix ComplexMethod debt and refactor code #5029

merged 2 commits into from Jul 6, 2022

Conversation

amitdash291
Copy link
Contributor

@amitdash291 amitdash291 commented Jul 2, 2022

This PR achieves the following:

  1. Fix ComplexMethod debt in UnnecessaryAbstractClass class
  2. Reduce the number of returns in methods of UnnecessaryAbstractClass class

@github-actions github-actions bot added the rules label Jul 2, 2022
@amitdash291 amitdash291 changed the title Fix ComplexCondition debt and refactor code Fix ComplexMethod debt and refactor code Jul 2, 2022
@amitdash291 amitdash291 changed the title Fix ComplexMethod debt and refactor code Fix ComplexMethod debt and refactor code Jul 2, 2022
@cortinico cortinico added this to the 1.21.0 milestone Jul 5, 2022
@BraisGabin BraisGabin added the housekeeping Marker for housekeeping tasks and refactorings label Jul 6, 2022
@BraisGabin BraisGabin merged commit 1a6d059 into detekt:main Jul 6, 2022
@amitdash291 amitdash291 deleted the fix-complex-condition-debt branch July 6, 2022 18:31
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 rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants