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.

Sign up for GitHub

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

Jump to bottom

9$: Warn on semicolon in if-else chain #265

Open

strager opened this issue May 5, 2021 · 1 comment · May be fixed by #290

Labels

for hire

Collaborator

strager commented May 5, 2021

The following code likely has a mistake:

if (x) {
  f();
} else; {  // mistake: stray semicolon
  g();
}

produce a warning for code like this.

strager added the for hire label

Collaborator Author

strager commented May 13, 2021

@someoneigna made a PR for this task. (GitHub won't let me assign them this task, though, because they haven't made a comment.) #290

strager linked a pull request

that will close this issue

#265 Warn on redundant semicolon after else #290

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment