Credence.Rule.NoRedundantEnumJoinSeparator
(credence v0.3.2)
Copy Markdown
Readability rule: Detects Enum.join("") and Enum.map_join("", mapper)
where the empty-string separator is passed explicitly.
Enum.join/1 and Enum.map_join/2 already default to "", so the argument
adds visual noise without changing behaviour.
Bad
graphemes |> Enum.join("")
Enum.join(list, "")
items |> Enum.map_join("", &to_string/1)
Enum.map_join(items, "", &to_string/1)Good
graphemes |> Enum.join()
Enum.join(list)
items |> Enum.map_join(&to_string/1)
Enum.map_join(items, &to_string/1)