Merge branch 'ssr-merge' into mli-merge

.github/workflows/build_docs.yml

@@ -32,6 +32,8 @@ on:
   push:
     branches:
       - develop
+    branches-ignore:
+      - smartsim-refactor
 
 jobs:
   build_docs:

.github/workflows/changelog.yml

@@ -30,6 +30,8 @@ name: enforce_changelog
 
 on:
   pull_request:
+    branches-ignore:
+      - smartsim-refactor
   push:
     branches:
       - develop

.github/workflows/run_tests.yml

@@ -69,6 +69,14 @@ jobs:
         with:
           python-version: ${{ matrix.py_v }}
 
+      - name: Check Test Files are Marked
+        run: |
+          diff <(find tests -path tests/_legacy -prune -o -type f -name 'test_*.py' -print \
+                 | xargs grep -l 'pytestmark' \
+                 | sort) \
+               <(find tests -path tests/_legacy -prune -o -type f -name 'test_*.py' -print \
+                 | sort)
+
       - name: Install build-essentials for Ubuntu
         if: contains( matrix.os, 'ubuntu' )
         run: |
@@ -131,8 +139,9 @@ jobs:
         run: |
           make check-mypy
 
-      - name: Run Pylint
-        run: make check-lint
+      # TODO: Re-enable static analysis once API is firmed up
+      # - name: Run Pylint
+      #   run: make check-lint
 
       # Run isort/black style check
       - name: Run isort
@@ -163,7 +172,7 @@ jobs:
         run: |
           echo "SMARTSIM_LOG_LEVEL=debug" >> $GITHUB_ENV
           py.test -s --import-mode=importlib -o log_cli=true --cov=$(smart site) --cov-report=xml --cov-config=./tests/test_configs/cov/local_cov.cfg --ignore=tests/full_wlm/ -m ${{ matrix.subset }} ./tests
-
+    
       # Upload artifacts on failure, ignoring binary files
       - name: Upload Artifact
         if: failure()

.pylintrc

@@ -167,7 +167,7 @@ max-module-lines=1000
 
 # Allow the body of a class to be on the same line as the declaration if body
 # contains single statement.
-single-line-class-stmt=no
+single-line-class-stmt=yes
 
 # Allow the body of an if to be on the same line as the test if there is no
 # else.