Skip to content
This repository was archived by the owner on Mar 13, 2018. It is now read-only.

added editorconfig #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sbmaxx
Copy link
Contributor

@sbmaxx sbmaxx commented Aug 27, 2014

Just added .editorconfig for convenience ;)

By default a lot of people has 4 spaces indentation, but in polymer — 2 spaces. Editorconfig can help in this difference.

@addyosmani
Copy link

Please note that at present, it doesn't appear like any of the other Polymer repos include an editorconfig file by default: https://github.com/Polymer/CustomElements and this may be rejected as team preference.

@sbmaxx
Copy link
Contributor Author

sbmaxx commented Aug 27, 2014

Including editorconfig in polymer-gestures will not break anything, but may be a signal to other projects begin to use it to make contributing easier. Also, polymer team can think about using JSCS for common codestyle.

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

Successfully merging this pull request may close these issues.

3 participants