Final formatting fixes after re-running all checks

- Re-ran black formatter after flake8 fixes reformatted 3 test files
- Verified flake8 still passes with no errors after black reformatting
- All 223 tests continue to pass
- Documentation builds with fewer warnings (19 vs 36)
- All essential checks now passing cleanly

Final status:
✅ Black formatting: Clean
✅ Flake8 linting: Clean
✅ All tests: 223 passing
✅ Documentation: Builds successfully

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: jeremymanning

docs/build/doctrees/environment.pickle

@@ -1,8 +1,8 @@
 Check Type,Command,Expected Result,Current Status,Notes/Commit Hash
-Black formatter,black clustrix/,No changes needed,complete,1 file reformatted (config.py)
-Black formatter tests,black tests/,No changes needed,complete,1 file reformatted (test_kubernetes_integration.py)
-Flake8 linter clustrix,flake8 clustrix/,No errors,complete,No errors found
-Flake8 linter tests,flake8 tests/,No errors,complete,Fixed 31 errors (unused imports and lambda assignments)
+Black formatter,black clustrix/,No changes needed,complete,No changes needed after re-run
+Black formatter tests,black tests/,No changes needed,complete,3 files reformatted after flake8 fixes
+Flake8 linter clustrix,flake8 clustrix/,No errors,complete,No errors found after re-run
+Flake8 linter tests,flake8 tests/,No errors,complete,No errors found after re-run
 Type checking,mypy clustrix/,No errors (if configured),blocked,45 errors found - extensive type annotation work needed
-All tests,pytest,All tests pass,complete,223 tests passed
-Documentation build,cd docs && make html,Docs build successfully,complete,Built with 36 warnings (mostly missing pages)
+All tests,pytest,All tests pass,complete,223 tests passed after re-run
+Documentation build,cd docs && make html,Docs build successfully,complete,Built with 19 warnings (down from 36)

docs/build/html/searchindex.js

@@ -180,7 +180,9 @@ def mock_execute_command(cmd):
 
         # Verify sbatch command was executed
         execute_calls = executor._execute_remote_command.call_args_list
-        sbatch_calls = [call_obj for call_obj in execute_calls if "sbatch" in str(call_obj)]
+        sbatch_calls = [
+            call_obj for call_obj in execute_calls if "sbatch" in str(call_obj)
+        ]
         assert len(sbatch_calls) > 0
 
     @patch("os.unlink")
@@ -463,6 +465,7 @@ def mock_get(remote_path, local_path):
         assert call_args[0] == "/tmp/test_job/result.pkl"  # remote path
         # Verify local path is a temporary file (cross-platform)
         import tempfile
+
         temp_dir = tempfile.gettempdir()
         assert call_args[1].startswith(temp_dir)  # local temp path
 

notes/all_checks_status.csv

@@ -2,6 +2,7 @@
 import time
 import multiprocessing as mp
 from concurrent.futures import ThreadPoolExecutor, ProcessPoolExecutor
+
 # Removed unused unittest.mock imports
 
 from clustrix.local_executor import (
@@ -257,6 +258,7 @@ def test_safe_pickle_test_success(self):
 
     def test_safe_pickle_test_failure(self):
         """Test pickle test with unpicklable objects."""
+
         # Lambda functions are not picklable
         def lambda_func(x):
             return x
@@ -266,6 +268,7 @@ def lambda_func(x):
 
         def unpicklable_func(x):
             return x + y
+
         assert _safe_pickle_test(unpicklable_func) is False
 
         # File objects are not picklable
@@ -279,6 +282,7 @@ def test_choose_executor_type_unpicklable_function(self):
 
         def unpicklable_func(x):
             return x + y
+
         result = choose_executor_type(unpicklable_func, (), {})
         assert result is True  # Should use threads
 

tests/test_executor.py

@@ -140,6 +140,7 @@ def function_without_loops():
 
     def test_detect_loops_source_unavailable(self):
         """Test loop detection when source code is unavailable."""
+
         # Lambda functions don't have accessible source code
         def no_source_func():
             return sum(range(10))