sindresorhus/editorconfig-sublime

Clean up overlaps with the `INI` package

Closed this issue · 0 comments

I installed EditorConfig alongside with the INI sublime package, and found that there is overlapping.

(Also, the EditorConfig package does not play well with A File Icon because of a hidden INI syntax -
as stated in SublimeText/AFileIcon#71 (comment)).

The most glaring issue is that both define a Set Syntax: EditorConfig. This is quite confusing - as the two syntaxes produce different colors. Which is in turn caused by different scope names.

I have compared the two, and the syntax from the INI package seems more developed - also stated in jwortmann/ini-syntax#16 (comment).

I propose two options to solve the issue:

  1. Update the EditorConfig syntax definition with the one in the INI package.
    After that, the INI package could drop the EditorConfig syntax definition.
    • This would require either updating the hidden INI syntax as well (to match)
    • or rather mark the INI package as a dependency - this is done with other packages as well
  2. Merge the two packages into one
    As this package has features missing from INI, like indent_style integration with SublimeText

I would prefer keeping this package separate and depending on INI for the INI syntax definition.
See more comparison at jwortmann/ini-syntax#16 (comment)

Either way, it would solve the duplicate command issue.