Skip to content
This repository has been archived by the owner on Apr 24, 2021. It is now read-only.

Support .editorconfig #3254

Open
seaneagan opened this issue Aug 25, 2014 · 5 comments
Open

Support .editorconfig #3254

seaneagan opened this issue Aug 25, 2014 · 5 comments
Milestone

Comments

@seaneagan
Copy link

It would be awesome to have support for .editorconfig (http://editorconfig.org).

Then, technically there would be no need for gui options for the related (i.e. whitespace) settings, but if those still existed, project-specific .editorconfigs would take precedence over them.

(related to #2729 and probably others)

@devoncarew
Copy link
Contributor

We're investigating using editorconfig - we weren't aware of it. It would be cool to piggyback on an existing format and convention instead of creating our own.

@devoncarew devoncarew added this to the Later milestone Aug 25, 2014
@brianblakely
Copy link

👍

@seaneagan
Copy link
Author

Presumably this would require implementing something similar to this in dart. For that purpose the following should be pretty helpful:

https://chromiumcodereview.appspot.com/506993004/
http://pub.dartlang.org/packages/ini

The ini package probably just needs a few tweaks to support [ and ] in section names, and to support # comments.

It would be nice to have this as an editorconfig (or editor_config ?) pub package for other uses.

@devoncarew
Copy link
Contributor

Yup, having a parser package for editorconfig would make sense. I didn't know about the glob package, but I believe quiver already has some glob support.

@seaneagan
Copy link
Author

Also it should use:

https://github.com/dart-lang/files.dart

to climb the file system looking for .editorconfigs and reading them and merging the props.

I may take a crack at this, will reply here if I do.

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

No branches or pull requests

4 participants