search

fortran-lang / fortls

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

Version to test #416

Open digrapsas opened 5 months ago

digrapsas commented 5 months ago

Hi @gnikit, the previous merge request closed automatically because I had to force merge the origin branch to keep my fork up to date.

The feature has been tested for a few weeks now on a daily basis on my project, it is working pretty well. Of course there are improvements that could be discussed, I bring some minor modifications from time to time. Let me know whenever you have time to check on this.

I haven't been able to check the issue with the CI tests, I have little experience on this but I'll try to have a quick look soon.

Regrards

digrapsas commented 5 months ago

@gnikit I when I type the command you proposed to update intrinsics,

python3 -m fortls.parsers.internal.intrinsics

I get the following error: RuntimeWarning: 'fortls.parsers.internal.intrinsics' found in sys.modules after import of package 'fortls.parsers.internal', but prior to execution of 'fortls.parsers.internal.intrinsics'; this may result in unpredictable behaviour

In the beginning I thought this was some issue with my dependencies, but I tried from another computer and on the mother branch, but still got the same issue. Let me know if you have any idea of how this could be fixed,

Regards

digrapsas commented 2 months ago

Hi @gnikit,

This is just a reminder of this activity waiting to be reviewed and eventually merged.

I have been testing the folding range since a couple of months now on a large scale industrial code, and it has been working without any issues.

Let me know if you have some time to take care of this,

Regards