search

SublimeLinter / SublimeLinter-haml-lint

SublimeLinter 3 plugin for Haml, using haml-lint.
MIT License
4 stars 8 forks source link

Update linter.py #1

Closed Yoshyn closed 9 years ago

Yoshyn commented 9 years ago

Apply the same syntax than rubocop-linter. Avoid generate something like : SublimeLinter: hamllint version query: /Users/yoshyn/.rbenv/shims/ruby /Users/yoshyn/.rbenv/shims/haml-lint --version

Yoshyn commented 9 years ago

Additional information : Your code work when using rvm but not using rbenv. I did not test the pull request code using rvm but since it's the same as rubocop-linter, i'm confident

jeroenj commented 9 years ago

What exactly does this fix? I'm no longer using Sublime Text. I've just checked both your code and master's code and I can't seem to find any difference.

I also use chruby (instead of rvm or rbenv) and it seems to work fine. Are you sure your path is set up correctly?

Yoshyn commented 9 years ago

It fix that he search hamllint on this path : /Users/yoshyn/.rbenv/shims/ruby /Users/yoshyn/.rbenv/shims/haml-lint --version Whereas I except he search on this path /Users/yoshyn/.rbenv/shims/ruby haml-lint --version

Using code like this linter https://github.com/SublimeLinter/SublimeLinter-rubocop (writing by SublimeLinter), it's work for me.

But i confess that i really don't know why it's working now and not before.

(Without concern : what do you use as replacement of sublime-text?)

jeroenj commented 9 years ago

I've merged your changes since they don't break anything and it is how the SublimeLinter team themselves do it. Thanks. :)

I've switched to vim (inside tmux) a few months ago.