Thanks to visit codestin.com
Credit goes to github.com

Skip to content

correct indent for oneline do: when moved to next line #241

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 26, 2015
Merged

correct indent for oneline do: when moved to next line #241

merged 1 commit into from
Aug 26, 2015

Conversation

tonini
Copy link
Contributor

@tonini tonini commented Aug 26, 2015

Example:

defmodule Hello do
  defp skip?(%{valid?: true, changes: empty, action: :update}) when empty == %{},
    do: true
  def so,
    do: hey
  defp so,
    do: asdasd
  defp so, do: hey
  defp so,
    do: hey
  defmacro so,
    do: hey
  defmacrop he, do: IO.puts "He"
end

tonini added a commit that referenced this pull request Aug 26, 2015
@tonini
Merge pull request #241 from elixir-lang/one-line-do-indentation
60aeb48 
correct indent for oneline `do:` when moved to next line
@tonini tonini merged commit 60aeb48 into master Aug 26, 2015
@tonini tonini deleted the one-line-do-indentation branch August 26, 2015 20:05
@michalmuskala
Copy link

This is great! ❤️

@gausby
Copy link
Contributor

gausby commented Aug 26, 2015

I second that!

@whatyouhide
Copy link

💟 😍 !!!

@jtmoulia
Copy link

🍻

J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
@msaraiva
Normalize diagnostic messages (elixir-editors#241)
5fbf589
J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
@axelson
Update changelog and remove known issue (elixir-editors#245)
ee532ee 
elixir-lsp/vscode-elixir-ls#93
elixir-lsp/elixir-ls#237
elixir-lsp/elixir-ls#227
elixir-lsp/elixir-ls#241

Removed known issue due to fix in elixir-editors#241
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tonini @michalmuskala @gausby @whatyouhide @jtmoulia