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

Semicolon after closing curly brace should yield an error #982

Closed
KSXGitHub opened this issue Sep 3, 2017 · 8 comments

Comments

@KSXGitHub
Copy link

commented Sep 3, 2017

if (foo) {
  console.log('bar')
}; // <-- Semicolon is here, but no errors
@tunnckoCore

This comment has been minimized.

Copy link

commented Sep 3, 2017

I seen that case and it was discussed already i believe. Don't remember what was the case and state.

@KSXGitHub

This comment has been minimized.

Copy link
Author

commented Sep 4, 2017

@charlike I couldn't believe if it was really intended behaviour. Semicolon after } doesn't fit anywhere, it doesn't fit the traditional C style, nor does it fit standard.js which in turn forbids semicolons.

@tunnckoCore

This comment has been minimized.

Copy link

commented Sep 4, 2017

I couldn't believe if it was really intended behaviour.

I didn't said that it is intended. I said that it was discussed and probably should be searched in the issues.

@KSXGitHub

This comment has been minimized.

Copy link
Author

commented Sep 4, 2017

@charlike I searched and found nothing relevant except this issue

image

@tunnckoCore

This comment has been minimized.

Copy link

commented Sep 4, 2017

Anyway.. it's needless to comment more. I just saying that i seen such discussion, so someone that remember more can link :)

I searched and found nothing relevant except this issue

I don't like the search very much. Almost always I don't found what i need so it's not factor for me :D

@stale

This comment has been minimized.

Copy link

commented May 10, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale label May 10, 2018

@stale stale bot closed this May 17, 2018

@feross

This comment has been minimized.

Copy link
Member

commented May 20, 2018

This may be fixable with the https://eslint.org/docs/rules/no-extra-semi rule. Re-opening this issue so I can take a look at fixing this in standard v12.

@feross feross reopened this May 20, 2018

@stale stale bot removed the stale label May 20, 2018

@feross feross added this to the standard v12 milestone May 20, 2018

@feross feross added the enhancement label May 20, 2018

@feross

This comment has been minimized.

Copy link
Member

commented May 20, 2018

Actually, closing as a duplicate of #786

@feross feross closed this May 20, 2018

@lock lock bot locked as resolved and limited conversation to collaborators Aug 18, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
3 participants
You can’t perform that action at this time.