Don't put multiple function defs together

[tallakt]

I think this is more like what you would find in elixir source code...

[Gazler]

The elixir code base seems to do the following for single line clauses:

def foo([]), do: true
def foo(nil), do: false
def foo(_), do
  #something else
end

As seen in https://github.com/elixir-lang/elixir/blob/2fb232a5318d8ea9db38dc2fc3dc81063a6b6304/lib/elixir/lib/list.ex#L322

And split multiple definitions with different clauses by a newline such as https://github.com/elixir-lang/elixir/blob/2fb232a5318d8ea9db38dc2fc3dc81063a6b6304/lib/elixir/lib/list.ex#L295 (as proposed here)

[tallakt]

Yeah thats probably more precise

[mschae]

Yeah I am in favour of keeping it as suggested right now.

I personally had to adjust to multiple function definitions not being separated, but once I got used to it it made reading code easier because it shows right away "what belongs together".

[jtmoulia]

👍 for what @Gazler said -- run together for single line clauses, split for multi-line clauses.

[nessamurmur]

👍 for what @Gazler said -- run together for single line clauses, split for multi-line clauses.

Another 👍.

Anyone want to open a PR for that?

[Gazler]

@niftyn8 @jtmoulia I have opened the pull request, would appreciate feedback on my addition of not mixing single-line and multi-line for more than one multi-line def.