search

editorconfig / editorconfig.github.com

Configuration file format for defining coding styles in shared projects
http://editorconfig.org
Other
266 stars 42 forks source link

Why are plugins required for PhpStorm and PyCharm? (but not for IntelliJ IDEA) #64

Closed bittner closed 5 years ago

bittner commented 8 years ago

The website states that plugins are required for the following JetBrains products

but not for the following JetBrains products

According to my knowledge all of them build on IntelliJ IDEA. In other words, if there is no plugin needed for IntelliJ IDEA there should be none needed for all other, dependent JetBrains IDEs. -- If this is true PhpStorm and PyCharm need to be moved up to the "No Plugin Necessary" list.

Update of JetBrains Plugin Description Needed?

In addition, JetBrains' plugin #7294 states to be "Compatible with: PhpStorm , AppCode , CLion , DataGrip , Android Studio". Looks like this needs to be clarified with JetBrains. (Why are PyCharm, RubyMine and WebStorm missing in the list?)

bittner commented 8 years ago

I've opened issue PY-20574 on YouTrack to clarify part 2 of this issue.

xuhdev commented 8 years ago

JetBrain maintains all of their plugins, so we'll just wait for their response...

denofevil commented 8 years ago

@bittner, EditorConfig plugin is bundled into WebStorm, RubyMine and IntelliJ IDEA. Thus, it's not needed for these IDEs. Other IDEs, such as PhpStorm and PyCharm don't bundle it, but it can be installed a plugin - that's why it's on the plugins site. I think only problem here is that CLion is shown in the first list, while it requires a plugin.

bittner commented 8 years ago

@denofevil Okay, so the website here needs to be updated. But the JetBrains plugin's description also needs to be updated then and include PyCharm, correct?

Interestingly though, PyCharm (2016.2.2) shows the "EditorConfig is overriding ..." message out-of-the-box, i.e. I didn't install any plugin myself. Was this shipped with the recent update?

denofevil commented 8 years ago

@bittner oops, sorry, looks like PyCharm is bundling it too, so it should be moved up

treyhunner commented 7 years ago

@denofevil so CLion should be moved down and PyCharm should be moved up, right?

denofevil commented 7 years ago

@treyhunner yes, that's right

xuhdev commented 7 years ago

Are we really going to make this change? This has been delayed for almost one year.

xuhdev commented 5 years ago

Seems fixed. Closed now.