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

Report if/else as issue location instead of block #5407

Merged
merged 1 commit into from Oct 13, 2022

Conversation

dizney
Copy link
Contributor

@dizney dizney commented Oct 12, 2022

Report mandatory braces issues on the actual if and/or else keywords instead of the then or else code blocks to prevent hiding potential other issues in those code blocks.

Closes #5317

Report mandatory braces issues on the actual if and/or else keywords
instead of the then or else code blocks to prevent hiding potential
other issues in those code blocks.

Closes detekt#5317
Copy link
Member

@BraisGabin BraisGabin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!!

@BraisGabin BraisGabin merged commit 48f5615 into detekt:main Oct 13, 2022
@dizney dizney deleted the report-location-mandatory-braces-if branch October 13, 2022 21:46
@BraisGabin BraisGabin added this to the 1.22.0 milestone Oct 14, 2022
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.

MandatoryBracesIfStatement to highlight only if/else keywords
2 participants