Do not check | in type hints in WPS226 (#2495)

* Do not check | in type hints in WPS226 * Update wemake_python_styleguide/visitors/ast/complexity/overuses.py * Update test_overused_string.py Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
wemake-services · Sep 23, 2022 · 45cd76a · 45cd76a
1 parent 8e7389e
commit 45cd76a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -32,6 +32,10 @@ Semantic versioning in our case means:
 - Domain name was changed from `wemake-python-stylegui.de`
   to `wemake-python-styleguide.rtfd.io`
 
+### Bugfixes
+
+- Fixes `WPS226` false positives on `|` use in `SomeType | AnotherType` type hints syntax
+
 
 ## 0.16.1
 

diff --git a/tests/test_visitors/test_ast/test_complexity/test_overuses/test_overused_string.py b/tests/test_visitors/test_ast/test_complexity/test_overuses/test_overused_string.py
@@ -177,6 +177,8 @@ def test_string_overuse(
     '"GenericType[int, str]"',
     '"int"',
     'List["int"]',
+    'list[int]',
+    'int | None',
 ])
 def test_string_type_annotations(
     assert_errors,

diff --git a/wemake_python_styleguide/visitors/ast/complexity/overuses.py b/wemake_python_styleguide/visitors/ast/complexity/overuses.py
@@ -48,6 +48,7 @@ class StringOveruseVisitor(base.BaseNodeVisitor):
         '\n',
         '\r\n',
         '\t',
+        '|',
         b' ',
         b'.',
         b',',
-Original file line number
+Diff line change
@@ Expand Up / @@ -48,6 +48,7 @@ class StringOveruseVisitor(base.BaseNodeVisitor): @@
             '\n',
             '\r\n',
             '\t',
+            '|',
             b' ',
             b'.',
             b',',
@@ Expand Down @@