Skip to content

Updates #274

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
Nov 9, 2021
Merged

Updates #274

merged 13 commits into from
Nov 9, 2021

Conversation

gnikit
Copy link
Member

@gnikit gnikit commented Nov 9, 2021

Ports code changes present in the formatting branch.
No controversial changes just cleaning and updating the base code.
Improved the warning system when useful tools are missing and other minor things.

This has now been replaced with `which` which is also platform agnostic.
The only difference is that `which` does not cache the result of the
query. However, realistically speaking calling which once per new
file being opened is a trivial cost.
Moreover, already opened files are cached automatically so no `which` calls
are made when changing between opened files.
We now prompt the user to install fortran-language-server and
Fortran IntelliSense if the packages are missing.

There is a slight issue with the calls being asynchronous, so if they are installed
in the wrong order Fortran IntelliSense first and then fortls an error
is being throw.
Fixes Rename fortran_fixed-form to FortranFixedForm #259
Fixes Extension does not activate for FixedForm #257
Fixes Linting does not work for fixed format files #258
1. Fixes conditional trigger for linting
2. Moves which as a dependency
3. Adds logging output for when parts of the extension are enabled
@gnikit gnikit self-assigned this Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant