search

REditorSupport / sublime-ide-r

R-IDE: Make Sublime Text a perfect IDE for R
MIT License
112 stars 7 forks source link

[Feature Request] Option for toggling the `lintr` on and off #15

Open amirmasoudabdol opened 5 years ago

amirmasoudabdol commented 5 years ago

I'm wondering if it's feasible to add an option to toggle lintr on and off. I understand that this is possible through setting specific options but it'd be nice to just have a toggle button, maybe even making accessible in the Command Pallate.

randy3k commented 5 years ago

Check the setting diagnostics.

randy3k commented 5 years ago

Oh. Didn’t read your post carefully. However, I am afraid that it is not possible to make it togglable. A restart is always needed.

amirmasoudabdol commented 5 years ago

I'd say that's fine. There are options on other packages, even on Sublime that states the necessary restart. It's already good if there is an option to toggle it on/off.

sckott commented 5 years ago

Seems the diagnostics only hides the bottom pane - but does not turn off/on the lint hints in the files. Is it possible to still have tab completion for functions/etc. but not linting advice?

randy3k commented 5 years ago

It should stop the entire linting feature. Try restarting the sublime

sckott commented 5 years ago

thanks, i think that works

divenyijanos commented 5 years ago

Check the setting diagnostics.

Could you please elaborate on this? I tried "diagnostics": false in my R-IDE.sublime-settings to turn off linting, but it does not seem to have an effect (not even after a restart).

randy3k commented 5 years ago

Are u using the latest version of languageserver? There is a bug in the previous version preventing the settings to pass over to the server.

divenyijanos commented 5 years ago

oh, thanks, I used 0.2.6, installing the latest version solved the issue

divenyijanos commented 5 years ago

Setting "diagnostics": false only disables linting or some other features too?

randy3k commented 5 years ago

It only disables linting.

richardli commented 4 months ago

Apologies replying to an old issue. But the above "diagnostics": false trick does not seem to be able to turn off lint for ST4. I'm using the latest build 4169 and latest version of languageserver. Many thanks!