Merge pull request #31 from gnikit/feature/coverage-improve

Improve code-base and coverage

Author: gnikit

.coveragerc

@@ -1,3 +1,10 @@
 [run]
 dynamic_context = test_function
 omit = fortls/__init__.py
+
+[report]
+exclude_lines =
+    if debug:
+    log.debug
+    except:
+    if not PY3K:

.github/workflows/main.yml

@@ -9,7 +9,7 @@ jobs:
   build:
     strategy:
       matrix:
-        os: [ubuntu-latest]
+        os: [ubuntu-latest, windows-latest]
         python-version: ["3.7", "3.8", "3.9", "3.10"]
       fail-fast: false
     runs-on: ${{ matrix.os }}

.github/workflows/python-publish.yml

@@ -55,18 +55,21 @@ jobs:
         shell: bash
         run: sed -i "s@\".*\"@\"${VERSION}\"@g" "fortls/_version.py"
 
-      - name: Commit the new version to dev
-        shell: bash
-        run: |
-          git config --global user.name 'gnikit'
-          git config --global user.email '[email protected]'
-          git fetch origin
-          git switch dev
-          git commit -S fortls/_version.py -m "Auto-Update version" -v
-          git push
-          git tag -f "${VERSION}"
-          git push --delete origin "${VERSION}"
-          git push origin "${VERSION}"
+      # Disabled the workflow because it messes up with the Releases on GitHub
+      # releases that use tags through force-push are marked as drafts
+      # will have to manually update the versions in _version.py
+      # - name: Commit the new version to dev
+      #   shell: bash
+      #   run: |
+      #     git config --global user.name 'gnikit'
+      #     git config --global user.email '[email protected]'
+      #     git fetch origin
+      #     git switch dev
+      #     git commit -S fortls/_version.py -m "Auto-Update version" -v
+      #     git push
+      #     git tag -f "${VERSION}"
+      #     git push --delete origin "${VERSION}"
+      #     git push origin "${VERSION}"
 
       - name: Build package
         run: python -m build

CHANGELOG.md

@@ -1,10 +1,19 @@
 # CHANGELONG
 
-## Unreleased
+## 2.1.0
 
 ### Added
 
 - Added coverage metric for Codecov
+- Added coverage for `WHERE`, `ENUM`, max line/comment diagnostics and multilines
+- Adds Windows CI
+
+### Fixed
+
+- Fixed global `sort_keywords` option not propagating during parsing on Windows
+  ([gnikit/fortls#36](https://github.com/gnikit/fortls/issues/36))
+- Fixed unittests not propagating debugger state
+  ([gnikit/fortls#35](https://github.com/gnikit/fortls/issues/35))
 
 ## 2.0.1
 

docs/conf.py

@@ -15,6 +15,8 @@
 
 sys.path.insert(0, os.path.abspath(".."))
 
+from fortls._version import __version__  # noqa: E402
+
 # Generate the agglomerated changes (from the CHANGELOG) between fortls
 # and the fortran-language-server project
 with open("../CHANGELOG.md", "r") as f:
@@ -57,7 +59,7 @@
 author = "Giannis Nikiteas"
 
 # The full version, including alpha/beta/rc tags
-release = "2.0.0"
+release = __version__
 
 
 # -- General configuration ---------------------------------------------------

docs/options.rst

@@ -84,7 +84,9 @@ source_dirs
 
 .. code-block:: json
 
-   "source_dirs": ["./**", "/external/fortran/src"]
+   {
+      "source_dirs": ["./**", "/external/fortran/src"]
+   }
 
 By default all directories under the current project will be recursively parsed
 for Fortran sources. Alternatively, one can define a series of directories
@@ -97,7 +99,9 @@ incl_suffixes
 
 .. code-block:: json
 
-   "incl_suffixes": [".h", ".FYP"]
+   {
+      "incl_suffixes": [".h", ".FYP"]
+   }
 
 ``fortls`` will parse only files with ``incl_suffixes`` extensions found in
 ``source_dirs``. By default ``incl_suffixes`` are defined as
@@ -112,7 +116,9 @@ excl_suffixes
 
 .. code-block:: json
 
-   "excl_suffixes": ["_tmp.f90", "_hdf5.F90"]
+   {
+      "excl_suffixes": ["_tmp.f90", "_hdf5.F90"]
+   }
 
 If certain files or suffixes do not need to be parsed these can be excluded by
 deffining ``excl_suffixes``
@@ -130,8 +136,9 @@ its subdirectories from being parsed you should define it like so
 
 .. code-block:: json
 
-   "excl_paths": ["exclude_dir/**"]
-
+   {
+      "excl_paths": ["exclude_dir/**"]
+   }
 
 Preprocessor
 ############
@@ -141,7 +148,9 @@ pp_suffixes
 
 .. code-block:: json
 
-   "pp_suffixes" : [".h", ".F90", ".fpp"]
+   {
+      "pp_suffixes" : [".h", ".F90", ".fpp"]
+   }
 
 By default preprocessor definitions are parsed for all Fortran source files
 with uppercase extensions e.g. ``.F90``, ``.F``, ``.F08``, etc.. However, the
@@ -153,7 +162,9 @@ include_dirs
 
 .. code-block:: json
 
-   "include_dirs": ["include", "preprocessor", "/usr/include"]
+   {
+      "include_dirs": ["include", "preprocessor", "/usr/include"]
+   }
 
 By default ``fortls`` will scan the project's directories for files with extensions
 ``PP_SUFFIXES`` to parse for **preprocessor definitions**. However, if the preprocessor
@@ -169,9 +180,11 @@ pp_defs
 
 .. code-block:: json
 
-   "pp_defs": {
-      "HAVE_PETSC": ""
-      "Mat": "type(tMat)"
+   {
+      "pp_defs": {
+         "HAVE_PETSC": ""
+         "Mat": "type(tMat)"
+      }
    }
 
 Additional **preprocessor definitions** from what are specified in files found in 

fortls.py

@@ -1,10 +1,6 @@
 #!/usr/bin/env python3
+# file used for unit testing
 if __name__ == "__main__":
-    import sys
-    import os
-
-    root_dir = os.path.abspath(os.path.join(os.path.dirname(__file__), ".."))
-    sys.path.insert(0, root_dir)
     import fortls
 
     fortls.main()

fortls/__init__.py

@@ -14,8 +14,8 @@
 from .interface import commandline_args
 
 
-def error_exit(error_str):
-    print("ERROR: {0}".format(error_str))
+def error_exit(error_str: str):
+    print(f"ERROR: {error_str}")
     sys.exit(-1)
 
 
@@ -25,7 +25,7 @@ def main():
     args = commandline_args(__name__).parse_args()
 
     if args.version:
-        print("{0}".format(__version__))
+        print(__version__)
         sys.exit(0)
 
     debug_server = (

fortls/_version.py

@@ -1 +1 @@
-__version__ = "v2.0.1"
+__version__ = "v2.1.0"

fortls/helper_functions.py

@@ -38,7 +38,7 @@ def expand_name(line: str, char_poss: int) -> str:
     regexs = [LOGICAL_REGEX, SQ_STRING_REGEX, DQ_STRING_REGEX, WORD_REGEX, NUMBER_REGEX]
     for r in regexs:
         for num_match in r.finditer(line):
-            if num_match.start(0) <= char_poss and num_match.end(0) >= char_poss:
+            if num_match.start(0) <= char_poss <= num_match.end(0):
                 return num_match.group(0)
     return ""
 
@@ -344,7 +344,7 @@ def get_keywords(keywords, keyword_info={}):
 def get_paren_substring(test_str):
     i1 = test_str.find("(")
     i2 = test_str.rfind(")")
-    if i1 > -1 and i2 > i1:
+    if -1 < i1 < i2:
         return test_str[i1 + 1 : i2]
     else:
         return None