Skip to content
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

Indent receive/after matches correct #259

Merged
merged 1 commit into from
Sep 16, 2015

Conversation

tonini
Copy link
Contributor

@tonini tonini commented Sep 16, 2015

Example:

receive do
  {:hello} -> :ok
  other ->
    other
after
  2000 ->
    IO.puts 'hello' # <- Correct indentation
    IO.puts 'status 2000 ends' # <- Correct indentation
  { :ok } ->
    IO.puts 'ok'
  _ -> whatever # <- Correct indentation
end

fixes #257

Example:

receive do
after
  2000 ->
    IO.puts 'hello' <- Indent two spaces
    IO.puts 'status 2000 ends' <- Additional lines too
  { :ok } ->
    IO.puts 'ok'
end
@tonini tonini force-pushed the correct-indent-for-receive-after branch from 07dd3e4 to 1cf6b1a Compare September 16, 2015 06:24
tonini added a commit that referenced this pull request Sep 16, 2015
…fter

Indent receive/after matches correct
@tonini tonini merged commit 7f9e218 into master Sep 16, 2015
@tonini tonini deleted the correct-indent-for-receive-after branch September 16, 2015 06:27
@beedub
Copy link

beedub commented Sep 16, 2015

thanks!

J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
* Remove keywords from completions

They were being returned regardless of context which made them very noisy.

Related:
* elixir-lsp/elixir_sense#99
* elixir-lsp/elixir-ls#251

* Add changelog entry

* Return isIncomplete = true for completions

* Fix tests and add a couple.
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.

receive/after (mis)indentation
2 participants