search

Jardinero / stylelint-plugin

Stylelint plugin for all products based on IntelliJ Platform (IntelliJ IDEA, RubyMine, WebStorm, PhpStorm, PyCharm, AppCode, etc.)
MIT License
20 stars 4 forks source link

Add support for Linux #3

Open catchin opened 7 years ago

catchin commented 7 years ago

The plugin searches for an executable called "stylelint.cmd". However, on Linux, it is only "stylelint". Could you add support also for this? One possible solution I could imagine would be to set the whole path to the executable in the environment variable and not only the directory where the executable is located. What do you think?

Jardinero commented 7 years ago

Hello! since this is my first plugin ever, I'm pretty much experimenting since the documentation isn't that good. This first approach was to get things working as fast as possible. For the portability issue you just mentioned I was planning on adding a config dialog in preference, so you may be able to select the executable, this would fix the issue you just mentioned. I'll keep you posted.

catchin commented 7 years ago

⁣​Hi, no problem, I appreciate your efforts. Config dialog sounds good. I noticed that there is already a config in the preferences where the stylelint node module location can be chosen. Don't know where this comes from though. If you need some support on this let me know.

Am 26. Nov. 2016, 23:42, um 23:42, Jardinero notifications@github.com schrieb:

Hello! since this is my first plugin ever, I'm pretty much experimenting since the documentation isn't that good. This first approach was to get things working as fast as possible. For the portability issue you just mentioned I was planning on adding a config dialog in preference, so you may be able to select the executable, this would fix the issue you just mentioned. I'll keep you posted.

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/Jardinero/stylelint-plugin/issues/3#issuecomment-263090359