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

Convert all indentation to either tabs or spaces? #26

Closed
ThePumpingLemma opened this issue Feb 25, 2014 · 2 comments
Closed

Convert all indentation to either tabs or spaces? #26

ThePumpingLemma opened this issue Feb 25, 2014 · 2 comments
Labels

Comments

@ThePumpingLemma
Copy link
Contributor

I noticed that a lot of the files have mixed indentation (i.e. both tabs and spaces). That leaves me with a few questions:

  • What is the official indentation policy? Tabs or spaces?
  • Can we run a script over the existing codebase to convert it to a single style?
  • Can we document the style so that contributors like me don't mess it up in the future? :)
@eugener
Copy link
Owner

eugener commented Feb 25, 2014

  1. The codebase was developed over several years from many different machines/environments. That is probably the reason for mixed indentation. I would personally prefer "all spaces" one.
  2. Yes. Would you be able to do it?
  3. Sure, I can add an article to the Wiki.

@eugener eugener closed this as completed Feb 25, 2014
@ThePumpingLemma
Copy link
Contributor Author

I'll submit a patch for it tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants