Add editorconfig #706

Closed
ftonato opened this Issue Oct 13, 2016 · 3 comments

Projects

None yet

3 participants

@ftonato
Contributor
ftonato commented Oct 13, 2016

EditorConfig helps developers define and maintain consistent coding styles between different editors and IDEs.

EditorConfig

# .editorconfig
root = true

[*]
indent_style = tab
indent_size = 8
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

[*.md]
trim_trailing_whitespace = false
@kvirani
Collaborator
kvirani commented Oct 13, 2016 edited

Totally agree. Though I believe it's 4 spaces atm. Want to submit a PR for this?

@ftonato
Contributor
ftonato commented Oct 13, 2016

I will submit a PR ;)

@ftonato ftonato added a commit to ftonato/velocity that referenced this issue Oct 13, 2016
@ftonato ftonato Add editorconfig. See #706 for more. e4590aa
@Rycochet
Collaborator
Rycochet commented Oct 14, 2016 edited

Spacing in the source is actually 1 tab right now - even though there's about 2x as many projects on GitHub using spaces as tabs - my personal view has always been "use tabs as then the coder can decide what spacing they want to display". Pretty much all of the arguments against tabs basically come back to ideas that were relevant over a decade ago ;-)

This is the sort of thing that is relevant for when we have automated testing and code style requirements - the argument about spaces / tabs would take up a large chunk of time ;-)

Probably relevant link: https://stackoverflow.com/questions/2316677/can-git-automatically-switch-between-spaces-and-tabs

@Rycochet Rycochet closed this Oct 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment