Skip to content

Commit 7359abb

Browse files
remyluslosiusclaude
andcommitted
fix(config): Standardize line length to 120 characters across all tools
Fixed line length inconsistency issue identified in quality assessment. Problem: - pyproject.toml configured for 100 characters - .pre-commit-config.yaml configured for 120 characters - scripts/quality-check.sh configured for 120 characters - Caused inconsistent formatting between tools Solution: Updated pyproject.toml to use 120 characters to match other tooling: - [tool.black] line-length: 100 → 120 - [tool.isort] line_length: 100 → 120 Rationale for 120 characters: ✅ Matches pre-commit hooks (.pre-commit-config.yaml) ✅ Matches quality-check.sh script ✅ 376 backend files already formatted to 120 chars (commit 54a7841) ✅ More readable on modern wide screens ✅ Industry best practice for Python (PEP 8 allows up to 99, but Black recommends 88-120 for modern development) Impact: - All formatting tools now use consistent 120-char line length - No more conflicts between local formatting and CI checks - Developers get consistent results across all quality tools Verified: - pyproject.toml: 120 ✓ - .pre-commit-config.yaml: 120 ✓ - scripts/quality-check.sh: 120 ✓ Closes: Critical Gaps Section 2 (Line Length Inconsistency) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent e0bd25b commit 7359abb

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

pyproject.toml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ python_classes = ["Test*"]
2727
python_functions = ["test_*", "validate_*"]
2828
markers = [
2929
"unit: Unit tests for individual components",
30-
"integration: Integration tests for component interactions",
30+
"integration: Integration tests for component interactions",
3131
"performance: Performance and load tests",
3232
"security: Security and compliance validation tests",
3333
"scap: SCAP scanning and processing tests",
@@ -50,7 +50,7 @@ filterwarnings = [
5050
source = ["app"]
5151
omit = [
5252
"*/tests/*",
53-
"*/validation/*",
53+
"*/validation/*",
5454
"*/migrations/*",
5555
"*/venv/*",
5656
"*/node_modules/*",
@@ -63,14 +63,14 @@ exclude_lines = [
6363
"pragma: no cover",
6464
"def __repr__",
6565
"raise AssertionError",
66-
"raise NotImplementedError",
66+
"raise NotImplementedError",
6767
"if __name__ == .__main__.:",
6868
"class .*\\bProtocol\\):",
6969
"@(abc\\.)?abstractmethod",
7070
]
7171

7272
[tool.black]
73-
line-length = 100
73+
line-length = 120
7474
target-version = ['py39', 'py310', 'py311']
7575
include = '\.pyi?$'
7676
extend-exclude = '''
@@ -91,7 +91,7 @@ extend-exclude = '''
9191

9292
[tool.isort]
9393
profile = "black"
94-
line_length = 100
94+
line_length = 120
9595
multi_line_output = 3
9696
include_trailing_comma = true
9797
force_grid_wrap = 0
@@ -113,4 +113,4 @@ warn_redundant_casts = true
113113
warn_return_any = true
114114
warn_unreachable = true
115115
warn_unused_configs = true
116-
warn_unused_ignores = true
116+
warn_unused_ignores = true

0 commit comments

Comments
 (0)