fix review

Author: xuhuanzy

crates/emmylua_code_analysis/src/compilation/analyzer/lua/closure.rs

@@ -163,10 +163,7 @@ fn analyze_return(
 
     signature.resolve_return = SignatureReturnStatus::InferResolve;
 
-    if returns.len() == 1 && returns[0].type_ref.is_nil() {
-    } else {
-        signature.return_docs = returns;
-    }
+    signature.return_docs = returns;
 
     Some(())
 }

crates/emmylua_code_analysis/src/db_index/type/humanize_type.rs

@@ -166,19 +166,15 @@ fn humanize_simple_type(
 }
 
 fn humanize_union_type(db: &DbIndex, union: &LuaUnionType, level: RenderLevel) -> String {
-    format_union_type(
-        union,
-        level,
-        |ty, level| humanize_type(db, ty, level.next_level()),
-        true,
-    )
+    format_union_type(union, level, |ty, level| {
+        humanize_type(db, ty, level.next_level())
+    })
 }
 
 pub fn format_union_type<F>(
     union: &LuaUnionType,
     level: RenderLevel,
     mut type_formatter: F,
-    is_hover: bool,
 ) -> String
 where
     F: FnMut(&LuaType, RenderLevel) -> String,
@@ -199,7 +195,7 @@ where
     let mut has_nil = false;
     let mut has_function = false;
     for ty in types.iter() {
-        if (!is_hover && !ty.is_function() && ty.is_nil()) || (is_hover && ty.is_nil()) {
+        if ty.is_nil() {
             has_nil = true;
             continue;
         } else if ty.is_function() {

crates/emmylua_ls/src/handlers/hover/hover_humanize.rs

@@ -485,12 +485,9 @@ fn hover_union_type(
     union: &LuaUnionType,
     level: RenderLevel,
 ) -> String {
-    format_union_type(
-        union,
-        level,
-        |ty, level| hover_type(builder, ty, Some(level)),
-        true,
-    )
+    format_union_type(union, level, |ty, level| {
+        hover_type(builder, ty, Some(level))
+    })
 }
 
 fn hover_multi_line_union_type(