Skip to content

Always indent with only 2 spaces except when function arguments span multiple lines. #149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 23, 2014

Conversation

tonini
Copy link
Contributor

@tonini tonini commented Dec 23, 2014

Always indent with only 2 spaces except when function arguments span multiple lines.

  • indent with 2 spaces
Enum.map(list, fn outer ->
  Enum.map(outer, fn inner ->
    some()
    code()
  end)
end)
  • exceptions
function(arg1, arg2,
         arg3)

Because this is a common community standard and also that's how most of the code in elixir source looks like, we think thats the proper way to do it from now on.

@tonini tonini force-pushed the indent-inside-parens branch from 349bc9f to 234dd51 Compare December 23, 2014 10:07
multiple lines.

* indent with 2 spaces

```
Enum.map(list, fn outer ->
  Enum.map(outer, fn inner ->
    some()
    code()
  end)
end)
```

* exceptions

```
function(arg1, arg2,
         arg3)
```
@tonini tonini force-pushed the indent-inside-parens branch from 234dd51 to abe5148 Compare December 23, 2014 10:07
tonini added a commit that referenced this pull request Dec 23, 2014
Always indent with only 2 spaces except when function arguments span multiple lines.
@tonini tonini merged commit 0f20892 into master Dec 23, 2014
@tonini tonini deleted the indent-inside-parens branch December 23, 2014 10:16
J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
* Get dialyzer warnings working in umbrella apps.

* Add helper functions after the function that needs them.
J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant