chore: check init.py in separate workflow step

RedHatInsights · Dec 4, 2023 · cef6a89 · cef6a89
1 parent 03f7be4
commit cef6a89
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -31,6 +31,9 @@ jobs:
         uses: actions/setup-python@v2
         with:
           python-version: ${{ matrix.python-version }}
+      - name: Check init.py
+        run: |
+          /bin/bash scripts/check_init_py.sh
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip poetry importlib-resources==1.5.0

diff --git a/run_tests.sh b/run_tests.sh
@@ -10,10 +10,6 @@ pip freeze > /tmp/req.txt
 pur -r /tmp/req.txt --verify True -o /dev/null | grep "Updated" | sed -e "s/Updated/You can update:/g"
 echo -e "${CYAN}------------------------------${NO_COLOR}"
 
-# Make sure there is __init__.py in each dir containing python scripts
-find . -name '*.py' | xargs dirname | sort | uniq | xargs -I {} bash -c "test -f {}/__init__.py || ( echo {} directory does not have __init__.py! && false )"
-rc=$(($rc+$?))
-
 # Run unit tests
 echo "Running unit tests:"
 pytest -vvv -s --cov-report term --cov --color=yes --durations=1

diff --git a/scripts/check_init_py.sh b/scripts/check_init_py.sh
@@ -0,0 +1,4 @@
+#!/bin/bash
+
+# Make sure there is __init__.py in each dir containing python scripts
+find . -name '*.py' | xargs dirname | sort | uniq | xargs -I {} bash -c "test -f {}/__init__.py || ( echo {} directory does not have __init__.py! && false )"