Merge pull request #8 from nasa/fix-ci-cd

This PR addresses initial repository setup issues and resolves linting errors to stabilize the development workflow.

- fixed existing linting violations in the codebase
- corrected formatting inconsistencies
- resolved issues with mypy static type checking
- ensured all CI checks now pass
- created a clean baseline for further development

Author: ank1m

.github/dependabot.yml

@@ -19,4 +19,4 @@ updates:
       gha-dependencies:
         patterns:
           - "*"
-    target-branch: "develop"
+    target-branch: "develop"

.github/pull_request_template.md

@@ -16,4 +16,4 @@ _Explain how this change was integration tested. Provide screenshots or logs if
 * [ ] Unit tests added/updated and passing.
 * [ ] Integration testing
 * [ ] `CHANGELOG.md` updated
-* [ ] Documentation updated (if needed).
+* [ ] Documentation updated (if needed).

.github/workflows/build-pipeline.yml

@@ -3,9 +3,7 @@ name: Build Harmony CASPER image
 
 # Controls when the workflow will run
 on:
-  # Triggers the workflow on push events
   push:
-    branches: [ develop, release/**, main, feature/**, issue/**, issues/** , docs/**]
 
   # Allows you to run this workflow manually from the Actions tab
   workflow_dispatch:
@@ -136,6 +134,7 @@ jobs:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
+
       - name: Extract metadata (tags, labels) for Docker
         if: ${{ !startsWith(github.ref, 'refs/heads/main/') }}
         id: meta
@@ -145,6 +144,7 @@ jobs:
           tags: |
             type=raw,pattern={{version}},value=${{ env.software_version }}
             type=raw,value=${{ env.venue }}
+
       - name: Build and push Docker image
         if: ${{ !startsWith(github.ref, 'refs/heads/main/') }}
         id: docker-push
@@ -154,7 +154,7 @@ jobs:
           file: Dockerfile
           build-args: |
             SOURCE=${{env.pyproject_name}}[harmony]==${{ env.software_version }}
-          push: true
+          push: false
           pull: true
           tags: ${{ steps.meta.outputs.tags }}
           labels: ${{ steps.meta.outputs.labels }}
@@ -168,4 +168,4 @@ jobs:
           git config user.name "${GITHUB_ACTOR}"
           git config user.email "${GITHUB_ACTOR}@users.noreply.github.com"
           git tag -a "${{ env.software_version }}" -m "Version ${{ env.software_version }}"
-          git push origin "${{ env.software_version }}"
+          git push origin "${{ env.software_version }}"

.github/workflows/pull_request.yml

@@ -1,8 +1,10 @@
-name: Release Branch Created
+name: Release Created
 
 # Run whenever a ref is created https://docs.github.com/en/actions/reference/events-that-trigger-workflows#create
 on:
-  create
+  push:
+    tags:
+      - '[0-9]+.[0-9]+.[0-9]+'
 
 env:
   POETRY_VERSION: "2.1.3"
@@ -39,4 +41,4 @@ jobs:
           git config --global user.name 'casper bot'
           git config --global user.email 'casper@noreply.github.com'
           git commit -am "/version ${{ env.software_version }}"
-          git push
+          git push

.github/workflows/release-created.yml

@@ -50,4 +50,4 @@ jobs:
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           verbose: true
-          flags: unittests
+          flags: unittests

.github/workflows/run_tests.yml

@@ -182,9 +182,9 @@ cython_debug/
 .abstra/
 
 # Visual Studio Code
-#  Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore 
+#  Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore
 #  that can be found at https://github.com/github/gitignore/blob/main/Global/VisualStudioCode.gitignore
-#  and can be added to the global gitignore or merged into this file. However, if you prefer, 
+#  and can be added to the global gitignore or merged into this file. However, if you prefer,
 #  you could uncomment the following to ignore the entire vscode folder
 # .vscode/
 

.gitignore

@@ -9,7 +9,7 @@ default_language_version:
 
 repos:
   - repo: https://github.com/gitleaks/gitleaks
-    rev: v8.28.0
+    rev: v8.30.0
     hooks:
       - id: gitleaks
 
@@ -33,18 +33,18 @@ repos:
       - id: end-of-file-fixer
       - id: mixed-line-ending
       - id: no-commit-to-branch # protects `main` by default
+        args: [--branch, main, --branch, develop, --pattern, ^release/.*$]
       - id: debug-statements
       - id: trailing-whitespace
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.14.3
+    rev: v0.14.10
     hooks:
       - id: ruff
         args: ["--fix", "--exit-non-zero-on-fix"]
       - id: ruff-format
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.18.2
+    rev: v1.19.1
     hooks:
       - id: mypy
-

.pre-commit-config.yaml

@@ -24,7 +24,7 @@ poetry install
 For example:
 
 ```shell
-poetry run casper TEMPO_NO2_L2_V04_S009G07.nc 
+poetry run casper TEMPO_NO2_L2_V04_S009G07.nc
 ```
 For example (_note that these are pseudo-real, not actual, TEMPO file names_):
 
@@ -93,4 +93,3 @@ Issues and pull requests welcome on [GitHub](https://github.com/nasa/harmony-cas
 ## License & Attribution
 
 CASPER is released under the [Apache License 2.0](http://www.apache.org/licenses/LICENSE-2.0).
-

README.md

@@ -1,25 +1,26 @@
-
 """A Harmony CLI wrapper around casper"""
 
 import logging
 import sys
 
 from casper.convert_to_csv import convert_to_csv
 from casper.file_ops import (
-      valid_input_file,
-      valid_workable_file,
+    valid_input_file,
+    valid_workable_file,
 )
 
+
 def run_casper(input_file: str):
     """Parse arguments and run casper on specified input file."""
     if not valid_input_file(input_file):
-            raise ValueError("Input filename not valid")
+        raise ValueError("Input filename not valid")
 
     if not valid_workable_file(input_file):
-            raise ValueError("Input file not valid")
+        raise ValueError("Input file not valid")
     zip_file_name = f"{input_file.split('/')[-1].split('.')[0]}.zip"
     convert_to_csv(input_file, zip_file_name)
 
+
 def main() -> None:
     """Entry point for the casper command line tool."""
     logging.basicConfig(
@@ -29,5 +30,6 @@ def main() -> None:
     )
     run_casper(sys.argv[1])
 
+
 if __name__ == "__main__":
     main()