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: allow semi as braceless body of statements (fixes #6386) #6391
Conversation
LGTM |
By analyzing the blame information on this pull request, we identified @vitorbal, @gyandeeps and @nzakas to be potential reviewers |
I missed the discussion- why avoid flagging the semicolon, rather than just avoiding a bad auto-fix? |
@platinumazure Cause if IfStatement doesn't have body, semi is required. And we are currently flagging it as extra. |
Ah, right. For some reason I thought we were recommending the use of empty
|
Thank you @alberto ! I'd like to fix similar issues in this, I guess |
Makes sense. I'll add those too. |
LGTM |
LGTM, thank you! |
No description provided.