Skip to content

Conversation

@floriangosse
Copy link
Contributor

No description provided.

@tunnckoCore
Copy link
Collaborator

you cant add what you want. @developit has preferences, and for #19 i was kinda forced to add editorconfig, so i added editorconfig that he want and thought to add :)

@developit
Copy link
Owner

developit commented Jan 18, 2017

Hmm - here I thought I was being clever using Github's web interface to edit, and it basically overwrote yours. Hopefully that's not annoying, sorry!

What you had was pretty close to what I usually use, just with some defaults moved around and a character set specified for JavaScript. The source for what I ammended is preact's editorconfig.

Let me know if my changes are alright, didn't mean to trample there!

@floriangosse
Copy link
Contributor Author

Everything okay. 😄

I didn't know about the preact's editorconfig. But it makes sense to reuse the config.

@developit
Copy link
Owner

Cool! Time to merge then :)

@developit developit merged commit d743bf8 into developit:master Jan 19, 2017
@floriangosse floriangosse deleted the patch-1 branch January 19, 2017 16:51
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

Successfully merging this pull request may close these issues.

3 participants