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

Please add a way to format sources without semicolons #264

Closed
alphara opened this issue Jan 17, 2017 · 5 comments
Closed

Please add a way to format sources without semicolons #264

alphara opened this issue Jan 17, 2017 · 5 comments
Labels
locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.

Comments

@alphara
Copy link

alphara commented Jan 17, 2017

Semicolons are excess in JavaScipt sources

@JustFly1984
Copy link

I agree that semicolons should be optional. Your tool is very nice, but currently we can't use it cos our style guides forbid semicolons.
Your tool adds to much stuff, and it is hard to diff commits, after your tool ads semicolons everywhere.

@viktorbezdek
Copy link

+1

Optional semicolons are a must for me. Your tool formats code really nicely otherwise.

@bnjmnt4n
Copy link
Contributor

This issue is a duplicate of #12. We won't be actively working on this feature, but if someone came up with a reasonable prototype/solution, we might work on adding this into Prettier.

@jlongster
Copy link
Member

Yes, please see the resolution at #12. You can freely code without semicolons but we will add them for consistently. It's not a simple matter of removing them because there are cases where we need to specially detect when to add them and it makes formatting harder.

This is an opinion that for now we are not going to change, much like standard will never have the option to add semicolons. Really, in the end, it doesn't matter at all and the fact that prettier will add them for you removes any annoyances you have in writing them.

@tony-kerz
Copy link

for me it's not as much about writing the semicolons as reading them, but i'm one of those guys that liked coffeescript too, so take that fwiw ;)

@lock lock bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jul 8, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Jul 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.
Projects
None yet
Development

No branches or pull requests

6 participants