Skip to content

Commit

Permalink
Merge pull request #2241 from sobolevn/patch-2
Browse files Browse the repository at this point in the history
`wemake` has 80 chars hard limit, not 79
  • Loading branch information
staticdev authored Jan 20, 2025
2 parents bced410 + 739634f commit b3d664f
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 3 deletions.
2 changes: 1 addition & 1 deletion isort/profiles.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
"multi_line_output": 3,
"include_trailing_comma": True,
"use_parentheses": True,
"line_length": 79,
"line_length": 80,
}
appnexus = {
**black,
Expand Down
16 changes: 14 additions & 2 deletions tests/unit/profiles/test_wemake.py
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,22 @@ class _ClassVisitor(ast.NodeVisitor): ...


def test_wemake_snippet_four():
"""80 line length should be fixed"""
"""80 line length should not be fixed"""
wemake_isort_test(
"""
from typing import Iterable, Iterator, Optional, Sequence, Tuple, TypeVar, Union
""",
"""
from typing import Iterable, Iterator, Optional, Sequence, Tuple, TypeVar, Union
""",
)


def test_wemake_snippet_five():
"""81 line length should be fixed"""
wemake_isort_test(
"""
from typing import Iterable, Iterator, Optional, Sequence, Tuple, TypeVar, Union1
""",
"""
from typing import (
Expand All @@ -102,7 +114,7 @@ def test_wemake_snippet_four():
Sequence,
Tuple,
TypeVar,
Union,
Union1,
)
""",
)

0 comments on commit b3d664f

Please sign in to comment.