Merge pull request #615 from mit-ll-responsible-ai/patch-coverage

Patch coverage and bump actions
mit-ll-responsible-ai · Dec 16, 2023 · 8d72b56 · 8d72b56
2 parents e9dfe57 + 9adc360
commit 8d72b56
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/pypi_publish.yml b/.github/workflows/pypi_publish.yml
@@ -22,7 +22,7 @@ jobs:
         pip install build
         python -m build
     - name: Upload artifacts
-      uses: actions/upload-artifact@v3
+      uses: actions/upload-artifact@v4
       with:
         name: dist
         path: dist
@@ -36,7 +36,7 @@ jobs:
       id-token: write  # IMPORTANT: this permission is mandatory for trusted publishing
     steps:
     - name: Download artifacts
-      uses: actions/download-artifact@v3
+      uses: actions/download-artifact@v4
       with:
         name: dist
         path: dist

diff --git a/src/hydra_zen/structured_configs/_type_guards.py b/src/hydra_zen/structured_configs/_type_guards.py
@@ -103,8 +103,7 @@ def is_old_partial_builds(x: Any) -> bool:  # pragma: no cover
             safe_getattr(x, "_partial_target_")
         ):
             return True
-        else:
-            # ensures we cover this branch in tests
+        else:  # pragma: no cover
             return False
     return False