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

Use 4-space indentation #23

Closed
aclissold opened this issue Aug 30, 2014 · 3 comments
Closed

Use 4-space indentation #23

aclissold opened this issue Aug 30, 2014 · 3 comments

Comments

@aclissold
Copy link
Contributor

It looks like fdb1ac8, which intended to fix auto-indentation, also decided to make 2 spaces the default. Was this intentional? I saw that this was decided against in issue #17, and 4 is a much less polarizing default than 2. Plus, the style guide mentioned in that issue is "centered on readability for print and the web."

@keith
Copy link
Owner

keith commented Aug 30, 2014

I think I did a git commit -a here when I was testing that out. I think that 2 spaces is the right way to go in the future. But right now, before the community agrees on that I've set it back to 4 21e44fa

@keith keith closed this as completed Aug 30, 2014
@aclissold
Copy link
Contributor Author

Sounds great! And thanks for this repo, I love it a lot 😄

@keith
Copy link
Owner

keith commented Aug 30, 2014

Thanks! If you notice any other issues please let me know!

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

2 participants