Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Add .editorconfig file #1332

Merged
merged 1 commit into from
Apr 30, 2018
Merged

Add .editorconfig file #1332

merged 1 commit into from
Apr 30, 2018

Conversation

Mr0grog
Copy link
Contributor

@Mr0grog Mr0grog commented Apr 30, 2018

(I made this as a PR rather than an issue because it’s so minimal. If we decide we don’t want it and just close the PR, that’s fine!)

As a follow-on to #1324, I thought it might be helpful to have a .editorconfig file in this repo. If you aren’t familiar with it, Editorconfig is a standardized file name & format for auto-configuring common text editor settings. It has built-in support in many editors and plug-in based support in pretty much all other major editors out there (the idea is you’d commit this instead of several configs for all the various editors contributors use). It’s a useful addition to a linter, since it silently and automatically configures your editor to do the right things, and is especially helpful when you work across several repos or languages that use different styles.

Disclosure: I maintain the TextMate plugin, so I may be biased on the value of this ;)

License: MIT
Signed-off-by: Rob Brackett <[email protected]>
@ghost ghost assigned Mr0grog Apr 30, 2018
@ghost ghost added the status/in-progress In progress label Apr 30, 2018
@Mr0grog Mr0grog requested a review from daviddias April 30, 2018 18:21
Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daviddias daviddias merged commit 993018f into master Apr 30, 2018
@daviddias daviddias deleted the chore/add-editorconfig branch April 30, 2018 19:32
@ghost ghost removed the status/in-progress In progress label Apr 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants