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

Semicolons #6

Closed
mdavidgreen opened this issue May 9, 2015 · 5 comments
Closed

Semicolons #6

mdavidgreen opened this issue May 9, 2015 · 5 comments

Comments

@mdavidgreen
Copy link
Contributor

Would you like some help adding semicolons to your code examples, or do you feel strongly about not using them? I notice you tend to leave them out in a lot of cases, but it's done inconsistently, so I wasn't sure what your intention was.

@syzer
Copy link
Contributor

syzer commented May 10, 2015

quite often return statement do not need a semicolon.

@DrBoolean
Copy link

I was thinking semicolons might just be a hassle while learning. Like putting make-up on a frog before dissecting it. That said, I have work to do to remove ones I subconsciously included...

@mdavidgreen
Copy link
Contributor Author

Students may be distracted when their linting tools with default settings tell them the code samples are full of errors due to missing semicolons. I'd be glad to help touch up your frog's lipstick and eyeliner, but I didn't want to presume, so let me know.

@DrBoolean
Copy link

Ahh, good thinking. I suppose they are necessary after all. I'm grateful for any help. Thanks a lot!

@shama
Copy link

shama commented Jul 20, 2015

My 2 bits. I have found semicolons to be confusing to people new to JavaScript as they are not obvious where they should and should not go. It is also perfectly valid to omit them and configure tooling to not warn when you prefer semicolons unwelcome.

Here are some additional opinions:

Also a shamelessly made video on the subject:

Although a good reason, IMO, to use semicolons is people will not get unreasonably angry at you if you do use semicolons but they oft do when you don't. In light, I'm not here to argue about semicolons but just inject my opinion on this open issue.

Thanks for this fantastic project!

@KtorZ KtorZ closed this as completed in ab1353b Feb 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants