search

fortran-lang / fortls

fortls - Fortran Language Server
https://fortls.fortran-lang.org/
MIT License
257 stars 41 forks source link

Fix passing `incl_suffixes` as command line arguments to fortls #299

Closed albertziegenhagel closed 1 year ago

albertziegenhagel commented 1 year ago

Passing --incl_suffix to fortls as command line arguments did not take any effect. Although the arguments were parsed correctly and set as attributes to the LangServer instance, they where never passed to src_file_exts and hence got ignored later when iterating the root directory.

This is especially problematic when fortls is used via the Modern Fortran VS code extension when it tries to pass along settings from the settings.json file.

codecov[bot] commented 1 year ago

Codecov Report

Merging #299 (c84e397) into master (c04d2ab) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #299   +/-   ##
=======================================
  Coverage   86.96%   86.96%           
=======================================
  Files          12       12           
  Lines        4571     4571           
=======================================
  Hits         3975     3975           
  Misses        596      596
Impacted Files Coverage Δ
fortls/langserver.py 84.53% <100.00%> (ø)