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

Spaces for indentation? #58

Closed
mauvm opened this issue Mar 7, 2015 · 5 comments
Closed

Spaces for indentation? #58

mauvm opened this issue Mar 7, 2015 · 5 comments

Comments

@mauvm
Copy link

mauvm commented Mar 7, 2015

I really like this forced standard! It's simple and will be effective. The only thing I can't get my head around is the 2 space indentation.

Why not use tabs instead? So that everyone in the team can configure their own horizontal depth?

@mauvm
Copy link
Author

mauvm commented Mar 7, 2015

Oh and sorry for being a b*tch about this. I just want to use this in my projects, but probably won't get everyone on board..

@max-mapper
Copy link
Contributor

@mauvm there are lots of debates online about tabs vs spaces. at the end of the day you have to 'just pick something'. thats the whole philosophy of standard -- its a bunch of sensible 'just pick something' opinions, and users will have to see the value in using that over defending their own opinions

@feross
Copy link
Member

feross commented Mar 7, 2015

Yep, what max said. :)

@feross feross closed this as completed Mar 7, 2015
@hugomallet
Copy link

For me this is a bad choice. But this debate will probably never end.
So why not keep this choice configurable? (maybe using ".editorconfig"?)

@dcposch
Copy link

dcposch commented Mar 26, 2015

Locking for the same reason as #78
If you care a lot about tabs vs spaces, standard isn't for you.
You can always use eslint directly.

@standard standard locked and limited conversation to collaborators Mar 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

5 participants