add pylint and flake

.flake8

@@ -0,0 +1,10 @@
+[flake8]
+# Ignore specific warnings or errors.
+ignore = E501
+
+# Optionally, increase the max line length (if you prefer to set a limit instead of ignoring entirely).
+max-line-length = 120
+
+# Exclude specific files or directories.
+exclude = tests/, docs/, .venv/
+

.github/workflows/deploy.yml

@@ -32,8 +32,11 @@ jobs:
           poetry build
           poetry install
 
-      - name: Run pytest
-        run: poetry run pytest
+      - name: Run Tests
+        run:|
+          poetry run flake8 podaac
+          poetry run pylint podaac
+          poetry run pytest
 
   # First job in the workflow installs and verifies the software
   deploy_sit:

.github/workflows/test_configs.yml

@@ -26,5 +26,8 @@ jobs:
           poetry build
           poetry install
 
-      - name: Run pytest
-        run: poetry run pytest
+      - name: Run Tests
+        run:|
+          poetry run flake8 podaac
+          poetry run pylint podaac
+          poetry run pytest

.pylintrc

@@ -0,0 +1,20 @@
+[MASTER]
+# List of directories to ignore (relative to the current directory).
+ignore=tests
+
+[MESSAGES CONTROL]
+# Disable the following messages by symbolic name or ID.
+disable=line-too-long,           # C0301: Line too long
+        broad-exception-caught,  # W0718: Catching too general exception Exception
+        unspecified-encoding,    # W1514: Using open without explicitly specifying an encoding
+        redefined-argument-from-local, # R1704: Redefining argument with a local name
+        no-value-for-parameter   # E1120: No value for argument in function call
+
+[FORMAT]
+# Maximum allowed line length (optional override if you want to enforce a slightly longer limit).
+max-line-length=120
+
+[REPORTS]
+# Disable the output of the report summary.
+reports=no
+

podaac/podaac_forge_tig_config_generator/generate_config.py

@@ -1,3 +1,5 @@
+"""script used to generate forge tig configuration"""
+
 import ast
 import json
 import os
@@ -7,7 +9,10 @@
 import pandas as pd
 from jsonschema import validate
 
+
 class HiTideConfigGenerator:
+    """Class to generate configurations"""
+
     REQUIRED_SETTINGS_SHEET = "required-settings"
     FORGE_PY_SHEET = "forge-py"
     TIG_SHEET = "tig"
@@ -126,6 +131,7 @@ def load_schema() -> Dict[str, Any]:
         with open(schema_path, "r") as file:
             return json.load(file)
 
+
 @click.command()
 @click.option('-f', '--file', help='Excel file with configuration settings', required=True)
 def generate_hitide_config(file: str):
@@ -149,5 +155,6 @@ def generate_hitide_config(file: str):
     except Exception as e:
         click.echo(f"Validation error: {e}", err=True)
 
+
 if __name__ == '__main__':
     generate_hitide_config()

pyproject.toml

@@ -16,6 +16,8 @@ jsonschema = "^4.23.0"
 pandas = "^2.2.3"
 openpyxl = "^3.1.5"
 click = "^8.1"
+pylint = "^3.3.2"
+flake8 = "^7.1.1"
 
 [build-system]
 requires = ["poetry-core"]