coding-standards icon indicating copy to clipboard operation
coding-standards copied to clipboard

Updates to IDE xml's - Version note them?

Open Hutchy68 opened this issue 10 years ago • 3 comments

Do we need to version out the IDE xml's? Or are we assuming every dev will have the latest version of their IDE of choice?

@betweenbrain you have https://github.com/betweenbrain/phpstorm-joomla-code-style/blob/master/Joomla.xml which works with the latest phpstorm? Or a specific version? For what version did @elkuku push.

There is also a PR version for fixes to Eclipse Kepler, assuming the current one for Eclipse is for Indigo. Current xml file is 2 years old.

Hutchy68 avatar Dec 21 '13 15:12 Hutchy68

I would assume that every dev is using the latest and greatest version of his/her IDE, so the XML files we are keeping here should be compatible with this version. I wouldn't put a specific version, but maybe something like "Tested with [IDE_NAME] version [VERSION_NO]" in the general readme, and maybe a link to a reference page (I just did that for PHPStorm here => #49)

I am afraid I can not remember which version of PHPStorm I was running when I submitted the XML file here, I guess it was about 3. I am currently running 7.1 and it still works ;) The file @betweenbrain has in his repo however looks much more "complete", containing also formatting rules for CSS and JavaScript - would be nice merging those ;)

elkuku avatar Dec 21 '13 19:12 elkuku

@elkuku thanks. I'd be happy to see them merged.

@hutchey68 that file should work with PhpStorm versions 5+, maybe earlier. I am currently using it with the latest version and all seems well. I do need to update o e small bit, but it should be fairly comprehensive.

I also don't think that we need to version them as they mostly tend to be backward compatible.

betweenbrain avatar Dec 22 '13 01:12 betweenbrain

Thanks guys, references to what works should be ok as a indicator. I think most update their IDE as they become available too.

Hutchy68 avatar Dec 22 '13 16:12 Hutchy68