Fix

Author: InSyncWithFoo

crates/ty_python_semantic/src/types/infer.rs

@@ -1136,28 +1136,29 @@ impl<'db, 'ast> TypeInferenceBuilder<'db, 'ast> {
 
                 kw_only_field_names.reverse();
 
-                if let (Some(first_field_name), other_fields) =
-                    (kw_only_field_names.pop(), kw_only_field_names)
-                {
-                    // TODO: The fields should be displayed in a subdiagnostic.
-                    if let Some(builder) = self
-                        .context
-                        .report_lint(&DUPLICATE_KW_ONLY, &class_node.name)
-                    {
-                        let mut diagnostic = builder.into_diagnostic(format_args!(
-                            "Dataclass has more than one field annotated with `KW_ONLY`"
-                        ));
+                match (kw_only_field_names.pop(), kw_only_field_names) {
+                    (Some(first_field_name), other_fields) if !other_fields.is_empty() => {
+                        // TODO: The fields should be displayed in a subdiagnostic.
+                        if let Some(builder) = self
+                            .context
+                            .report_lint(&DUPLICATE_KW_ONLY, &class_node.name)
+                        {
+                            let mut diagnostic = builder.into_diagnostic(format_args!(
+                                "Dataclass has more than one field annotated with `KW_ONLY`"
+                            ));
 
-                        diagnostic
-                            .info(format_args!("First `KW_ONLY` field: `{first_field_name}`"));
-                        diagnostic.info(format_args!(
-                            "Other fields: {}",
-                            other_fields
-                                .iter()
-                                .map(|name| format!("`{name}`"))
-                                .join(", ")
-                        ))
+                            diagnostic
+                                .info(format_args!("First `KW_ONLY` field: `{first_field_name}`"));
+                            diagnostic.info(format_args!(
+                                "Other fields: {}",
+                                other_fields
+                                    .iter()
+                                    .map(|name| format!("`{name}`"))
+                                    .join(", ")
+                            ));
+                        }
                     }
+                    _ => {}
                 }
             }
         }