Comment fixes

Signed-off-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>

Author: ericspod

.github/workflows/cicd_tests.yml

@@ -69,8 +69,15 @@ jobs:
       shell: bash
       run: |
         echo "skip=1" >> $GITHUB_OUTPUT
+
         mkdir -p /tmp/check_changes
-        git diff --name-only HEAD^..HEAD ':!docs' | while read i
+        if ! git diff --name-only HEAD^..HEAD ':!docs' > /tmp/check_changes/changed_files
+        then
+          echo "skip=0" >> "$GITHUB_OUTPUT"
+          exit 0
+        fi
+
+        while read i
         do
           echo "Checking for changes: $i"
           if ! git show HEAD^:$i > /tmp/check_changes/$(basename $i)
@@ -85,7 +92,7 @@ jobs:
 
           echo "skip=0" >> $GITHUB_OUTPUT
           break
-        done
+        done < /tmp/check_changes/changed_files
 
   static-checks:  # Perform static type and other checks using runtests.sh
     runs-on: ubuntu-latest

docs/source/utils.rst

@@ -87,6 +87,6 @@ Safe Evaluation
   :members:
 
 Compare Sources
---------------=
+---------------
 .. automodule:: monai.utils.compare_sources
   :members:

monai/utils/compare_sources.py

@@ -16,10 +16,11 @@
 from __future__ import annotations
 
 import ast
+import os
 import sys
 from os.path import splitext
 
-from monai.config.type_definitions import PathLike
+PathLike = str | os.PathLike  # needs to be duplicated here to avoid importing anything from MONAI inside actions
 
 SKIP_EXTS = (".md", ".rst")
 
@@ -54,8 +55,10 @@ def sources_equal(src1: str, src2: str) -> bool:
     """
     remdoc = RemoveDocstrings()
 
-    m1: ast.Module = remdoc.generic_visit(ast.parse(src1))
-    m2: ast.Module = remdoc.generic_visit(ast.parse(src2))
+    m1 = ast.parse(src1)
+    m2 = ast.parse(src2)
+    remdoc.visit(m1)
+    remdoc.visit(m2)
 
     list1 = list(ast.walk(m1))
     list2 = list(ast.walk(m2))

tests/utils/test_compare_sources.py

@@ -48,7 +48,19 @@ def method(self,arg1):
         return arg1
 """
 
-TEXT_EQUAL_PAIRS = [(text1, text2), (text1, text3), (text2, text3), ("", "")]
+text6 = """
+'''
+This is a module containing only a docstring.
+'''
+"""
+
+text7 = """
+'''
+So is this but it's different.
+'''
+"""
+
+TEXT_EQUAL_PAIRS = [(text1, text2), (text1, text3), (text2, text3), (text6, text7), (text6, ""), ("", "")]
 
 TEXT_UNEQUAL_PAIRS = [(text1, text4), (text2, text4), (text3, text4), (text1, text5), (text1, "")]