Skip to content

Commit 3a349f9

Browse files
fix: resolve all ruff lint errors
Remove unused imports (sys, Path, field, MappedEdge, ViolatingEdge, calculate_distance_metrics_for_project), fix import sorting, remove extraneous f-string prefixes. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 95c0b3a commit 3a349f9

File tree

10 files changed

+22
-26
lines changed

10 files changed

+22
-26
lines changed

src/archunitpython/__init__.py

Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3,26 +3,25 @@
33
__version__ = "0.1.0"
44

55
# Files API
6-
from archunitpython.files import project_files, files
7-
8-
# Slices API
9-
from archunitpython.slices import project_slices
10-
11-
# Metrics API
12-
from archunitpython.metrics import metrics
13-
14-
# Testing
15-
from archunitpython.testing import assert_passes, format_violations
16-
176
# Common
187
from archunitpython.common import (
19-
Violation,
20-
EmptyTestViolation,
218
CheckOptions,
9+
EmptyTestViolation,
2210
TechnicalError,
2311
UserError,
12+
Violation,
2413
)
25-
from archunitpython.common.extraction import extract_graph, clear_graph_cache
14+
from archunitpython.common.extraction import clear_graph_cache, extract_graph
15+
from archunitpython.files import files, project_files
16+
17+
# Metrics API
18+
from archunitpython.metrics import metrics
19+
20+
# Slices API
21+
from archunitpython.slices import project_slices
22+
23+
# Testing
24+
from archunitpython.testing import assert_passes, format_violations
2625

2726
__all__ = [
2827
# Files

src/archunitpython/common/assertion/violation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
from __future__ import annotations
44

5-
from dataclasses import dataclass, field
5+
from dataclasses import dataclass
66
from typing import Any
77

88

src/archunitpython/common/extraction/extract_graph.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@
44

55
import ast
66
import os
7-
import sys
8-
from pathlib import Path
97

108
from archunitpython.common.extraction.graph import Edge, Graph, ImportKind
119
from archunitpython.common.fluentapi.checkable import CheckOptions

src/archunitpython/common/logging/types.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
from __future__ import annotations
44

5-
from dataclasses import dataclass, field
5+
from dataclasses import dataclass
66
from typing import Literal
77

88
LogLevel = Literal["debug", "info", "warn", "error"]

src/archunitpython/common/projection/project_edges.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from collections import defaultdict
66

77
from archunitpython.common.extraction.graph import Edge
8-
from archunitpython.common.projection.types import MapFunction, MappedEdge, ProjectedEdge
8+
from archunitpython.common.projection.types import MapFunction, ProjectedEdge
99

1010

1111
def project_edges(

src/archunitpython/metrics/common/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
EnhancedClassInfo,
44
FieldInfo,
55
FileAnalysisResult,
6+
MethodInfo,
67
Metric,
78
MetricComparison,
8-
MethodInfo,
99
)
1010

1111
__all__ = [

src/archunitpython/metrics/extraction/extract_class_info.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import ast
66
import os
77

8-
from archunitpython.common.extraction.extract_graph import _find_python_files, _DEFAULT_EXCLUDE
8+
from archunitpython.common.extraction.extract_graph import _DEFAULT_EXCLUDE, _find_python_files
99
from archunitpython.metrics.common.types import (
1010
ClassInfo,
1111
EnhancedClassInfo,

src/archunitpython/metrics/fluentapi/metrics.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
StatementCountMetric,
3131
)
3232
from archunitpython.metrics.calculation.distance import (
33-
calculate_distance_metrics_for_project,
3433
calculate_file_distance_metrics,
3534
)
3635
from archunitpython.metrics.calculation.lcom import (
@@ -251,11 +250,12 @@ def __init__(self, project_path, filters, metric, threshold, comparison) -> None
251250
self._comparison: MetricComparison = comparison
252251

253252
def check(self, options: CheckOptions | None = None) -> list[Violation]:
253+
import os
254+
254255
from archunitpython.common.extraction.extract_graph import (
255256
_DEFAULT_EXCLUDE,
256257
_find_python_files,
257258
)
258-
import os
259259

260260
project = self._project_path or os.getcwd()
261261
project = os.path.abspath(project)

src/archunitpython/slices/fluentapi/slices.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
from archunitpython.slices.assertion.admissible_edges import (
2020
CoherenceOptions,
2121
Rule,
22-
ViolatingEdge,
2322
gather_positive_violations,
2423
gather_violations,
2524
)

src/archunitpython/testing/common/violation_factory.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ def from_violation(violation: Violation) -> TestViolation:
3737

3838
if isinstance(violation, ViolatingNode):
3939
return TestViolation(
40-
message=f"File pattern violation",
40+
message="File pattern violation",
4141
details=f"File '{violation.projected_node.label}' "
4242
f"{'matches' if violation.is_negated else 'does not match'} "
4343
f"pattern '{violation.check_pattern.regexp.pattern}'",
@@ -46,7 +46,7 @@ def from_violation(violation: Violation) -> TestViolation:
4646
if isinstance(violation, ViolatingFileDependency):
4747
edge = violation.dependency
4848
return TestViolation(
49-
message=f"File dependency violation",
49+
message="File dependency violation",
5050
details=f"'{edge.source_label}' "
5151
f"{'depends on' if violation.is_negated else 'does not depend on'} "
5252
f"'{edge.target_label}'",

0 commit comments

Comments
 (0)