search

AtomLinter / linter-julia

This linter plugin provides an interface to Lint.jl.
https://atom.io/packages/linter-julia
MIT License
17 stars 3 forks source link

Linter spams all cpu, making my laptop unworkable #114

Closed schlichtanders closed 3 years ago

schlichtanders commented 4 years ago

Hi,

I just gave linter-julia another try, because I really miss the linting in Atom. It's aweful unfortunately. Not only does it not work, but it blocks all CPU nodes, on my powerful Mac.

I hope someone in the future, this will just work, hence I open this ticket for someone to be able to address it.

This screenshot is done while atom is still open:

Screenshot 2020-06-13 at 15 18 57

This screenshot is done after atom was quit:

Screenshot 2020-06-13 at 15 22 49

After about 5 min, all processes seemed to be stopped finally...

As you can see, it is not useable and not recommendable. I hope that somewhen it will work.

Unfortunately I cannot tell you which julia version was used. Usually I have julia 1.4.1 currently running in Juno and on commandline, however no linter package seemed to be installed there when I check the status. So maybe some other julia version (I have a couple installed)...

OS: Mac Catalina

schlichtanders commented 4 years ago

I now installed ] add https://github.com/tonyhffong/Lint.jl in my julia 1.4.1 and hardcoded the julia path in the vscode setting of linter-julia, however still same results...

aminya commented 4 years ago

Please do not use the linter. Lint.jl is not working, and we have not found a way to integrate StaticLint.jl

https://github.com/julia-vscode/StaticLint.jl/issues/14 https://github.com/julia-vscode/StaticLint.jl/issues/37

I don't know how to use StaticLint. The documentation is missing

schlichtanders commented 4 years ago

thanks for the notice, still cost me 2 hours today.

Please, if you mind, update the official extension page https://atom.io/packages/linter-julia with big fat CURRENTLY NOT WORKING.

This is really annoying to be suggested it may work and then find that your complete system goes crazy.

schlichtanders commented 4 years ago

I took a look at the link you send, it says there is no API, however the front-page of StaticLint.jl actually has a longer explanation https://github.com/julia-vscode/StaticLint.jl

aminya commented 4 years ago

@schlichtanders I will try to implement the language server and update this package. Stay tuned.

schlichtanders commented 4 years ago

thank you so much

please also update the extension page https://atom.io/packages/linter-julia respectively with a warning, it is so annoying you have to understand.

aminya commented 4 years ago

OK. I added a warning to the readme. I will publish it to atom.io aswell

aminya commented 3 years ago

Fixed in #119