Skip to content

Commit 3e3f017

Browse files
jeremymanningclaude
andcommitted
Fix linting issues: remove trailing whitespace and organize imports
- Remove trailing whitespace from azure.py line 93 - Fix whitespace in notebook_magic.py docstring comments - Reorganize imports in test_widget_fixes.py to comply with flake8 All checks now pass: - flake8: 0 issues - pytest: 312/312 tests passing - mypy: 0 issues 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 41c0395 commit 3e3f017

File tree

3 files changed

+12
-9
lines changed

3 files changed

+12
-9
lines changed

clustrix/cloud_providers/azure.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ def authenticate(self, **credentials) -> bool:
9090
assert isinstance(client_id, str)
9191
assert isinstance(client_secret, str)
9292
assert isinstance(subscription_id, str)
93-
9493
self.credential = ClientSecretCredential(
9594
tenant_id=tenant_id, client_id=client_id, client_secret=client_secret
9695
)

clustrix/notebook_magic.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ class widgets: # type: ignore
187187
logger = logging.getLogger(__name__)
188188

189189
#: Default cluster configurations available in the widget.
190-
#:
190+
#:
191191
#: This dictionary contains pre-configured cluster templates for common use cases.
192192
#: Each configuration is a dictionary with cluster-specific settings.
193193
DEFAULT_CONFIGS = {
@@ -2367,13 +2367,13 @@ class ClusterfyMagics(Magics):
23672367
def clusterfy(self, line, cell):
23682368
"""
23692369
Create an interactive widget for managing Clustrix configurations.
2370-
2370+
23712371
Usage::
2372-
2372+
23732373
%%clusterfy
2374-
2374+
23752375
This creates a widget interface that allows you to:
2376-
2376+
23772377
- Select and manage cluster configurations
23782378
- Create new configurations with validation
23792379
- Save/load configurations from files

tests/test_widget_fixes.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,17 @@
22
Test suite for widget configuration fixes addressing issue #53.
33
"""
44

5-
import pytest
65
import importlib
6+
77
import clustrix.notebook_magic
8+
from clustrix.config import ClusterConfig
89

10+
# Reload to ensure fresh state
911
importlib.reload(clustrix.notebook_magic)
10-
from clustrix.notebook_magic import DEFAULT_CONFIGS, ClusterConfigWidget
11-
from clustrix.config import ClusterConfig
12+
13+
# Import after reload to get the refreshed module
14+
DEFAULT_CONFIGS = clustrix.notebook_magic.DEFAULT_CONFIGS
15+
ClusterConfigWidget = clustrix.notebook_magic.ClusterConfigWidget
1216

1317

1418
class TestWidgetConfigurationFixes:

0 commit comments

Comments
 (0)