Skip to content
New issue

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

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

Already on GitHub? # to your account

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

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant