fortls icon indicating copy to clipboard operation
fortls copied to clipboard

mention eglot for emacs support

Open ssnnoo opened this issue 2 years ago • 7 comments

eglot is a popular lsp client for emacs that supports fortls.

ssnnoo avatar May 24 '22 20:05 ssnnoo

Codecov Report

Merging #129 (447c6af) into master (34f5bbf) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 447c6af differs from pull request most recent head 6053c22. Consider uploading reports for the commit 6053c22 to get more accurate results

@@           Coverage Diff           @@
##           master     #129   +/-   ##
=======================================
  Coverage   86.00%   86.00%           
=======================================
  Files          11       11           
  Lines        4430     4430           
=======================================
  Hits         3810     3810           
  Misses        620      620           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov[bot] avatar May 25 '22 09:05 codecov[bot]

Ok, I will update the pull request tonight.

Regarding the wrong link please see https://github.com/joaotavora/eglot/pull/965, hopefully this will be updated soon.

Last but not least: if in f90 mode, eglot will start fortls as default, so no copy and paste required :-)

ssnnoo avatar May 25 '22 10:05 ssnnoo

Ok, I updated the pull request.

Unfortunately, nothing has happend regarding https://github.com/joaotavora/eglot/pull/965 yet.

ssnnoo avatar May 31 '22 13:05 ssnnoo

Any news on this @ssnnoo ?

gnikit avatar Jul 12 '22 12:07 gnikit

Any news on this @ssnnoo ?

Hi, I changed the commit as requested a while ago and it should be fine now (hopefully). One thing that did not happen is that https://github.com/joaotavora/eglot/pull/965 got merged. But this is completely independent of this contribution here :-)

ssnnoo avatar Jul 12 '22 14:07 ssnnoo

Any news on this @ssnnoo ?

Hi, I changed the commit as requested a while ago and it should be fine now (hopefully). One thing that did not happen is that joaotavora/eglot#965 got merged. But this is completely independent of this contribution here :-)

The problem is that the underlying extension is pointing to the wrong server, which leads to confusion from the user's perspective. I will go drop a comment

gnikit avatar Jul 12 '22 17:07 gnikit

Any news on this @ssnnoo ?

Hi, I changed the commit as requested a while ago and it should be fine now (hopefully). One thing that did not happen is that joaotavora/eglot#965 got merged. But this is completely independent of this contribution here :-)

The problem is that the underlying extension is pointing to the wrong server, which leads to confusion from the user's perspective. I will go drop a comment

Sure, I agree. But this is only in the readme, in the source code it only refers to fortls.

((fortran-mode f90-mode) . ("fortls")) and this should be fine. There is not automatic installation or anything.

ssnnoo avatar Jul 12 '22 17:07 ssnnoo