ardcore / atom-language-settings

MIT License
8 stars 2 forks source link

merge with cam-stitt/atom-syntax-settings #6

Open ahti opened 9 years ago

ahti commented 9 years ago

cam-stitt/atom-syntax-settings and this package seem to try to achieve very similar goals. I just went to search for a package like this and don't see any fundamental difference between the two.

Maybe it is worth considering merging the two into one package?

@ardcore @cam-stitt

cam-stitt commented 9 years ago

There is actually quite a big difference in the way the settings are provided. I don't override any config objects directly. Instead I modify the editor/editorview directly via their api methods. This package temporarily modifies the atom config each time they open a new file, and then resets that config when the file is closed. Both packages work great, but there are some pretty fundamental differences.