Fix "line-too-long" for property of model (#2837)

* fix line-too-long for property of model * changelog
Azure · Sep 12, 2024 · 1670cc6 · 1670cc6
1 parent 2b29fe0
commit 1670cc6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/.chronus/changes/fix-pylint-2024-09-12-2024-8-12-16-9-4.md b/.chronus/changes/fix-pylint-2024-09-12-2024-8-12-16-9-4.md
@@ -0,0 +1,7 @@
+---
+changeKind: fix
+packages:
+  - "@azure-tools/typespec-python"
+---
+
+Fix "line-too-long" for property of model
diff --git a/packages/typespec-python/generator/pygen/codegen/serializers/model_serializer.py b/packages/typespec-python/generator/pygen/codegen/serializers/model_serializer.py
@@ -261,9 +261,10 @@ def declare_property(prop: Property) -> str:
             else ""
         )
         generated_code = f'{prop.client_name}: {prop.type_annotation()} = {field}({", ".join(args)})'
+        # there is 4 spaces indentation so original line length limit 120 - 4 = 116
         pylint_disable = (
             " # pylint: disable=line-too-long"
-            if len(generated_code) <= 120 < (len(generated_code) + len(type_ignore))
+            if len(generated_code) <= 116 < (len(generated_code) + len(type_ignore))
             else ""
         )
         return f"{generated_code}{type_ignore}{pylint_disable}"