Removed linting step from CI. Removved linting comments. Removed lint…

…ing dependencies. (#169)
ACCESS-NRI · Dec 4, 2024 · 7f21734 · 7f21734
1 parent cc0f2ba
commit 7f21734
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 8 deletions.
diff --git a/.conda/env_dev.yml b/.conda/env_dev.yml
@@ -21,6 +21,4 @@ dependencies:
     - nccmp
     - pip
     - pytest
-    - pytest-cov
-    - pylint #TODO: remove this after setting up linting with ruff
-    - ruff
+    - pytest-cov
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -97,10 +97,6 @@ jobs:
         shell: bash -l {0}
         run: conda list
 
-      - name: Lint
-        shell: bash -l {0}
-        run: pylint --extension-pkg-whitelist=netCDF4 --ignored-modules=um2nc -E um2nc
-
       - name: Entrypoint test of driver script
         shell: bash -l {0}
         run: esm1p5_convert_nc --help

diff --git a/src/um2nc/conversion_driver_esm1p5.py b/src/um2nc/conversion_driver_esm1p5.py
@@ -179,7 +179,7 @@ def get_ff_date(fields_file_path):
     header = mule.FixedLengthHeader.from_file(
                                             str(fields_file_path))
 
-    return header.t2_year, header.t2_month, header.t2_day  # pylint: disable=no-member
+    return header.t2_year, header.t2_month, header.t2_day
 
 
 def convert_fields_file_list(input_output_paths):