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

correct indentation for block with multiple matches #231

Merged
merged 1 commit into from
Aug 4, 2015

Conversation

tonini
Copy link
Contributor

@tonini tonini commented Aug 4, 2015

fixes #227

This PR refines the indentation for block of matches.

case parse do
  { [ help: true ], _, _ }
    -> :help
  { _, [ user, project, count ], _ } ->
    { user, project, count }
  { _, [ user, project ], _ } -> { user, project, @default_count }
  _ -> :help
end

case parse do
  { [ help: true ], _, _ }
    -> :help
  { _, [ user, project, count ], _ }
    -> { user, project, count }
  { _, [ user, project ], _ }
    -> { user, project, @default_count }
  _ -> :help
  asd -> asda
  asdasd ->
    asd
  asd ->
    asd
    asd
  asdasd -> asdad
  asdasd ->
    asdasd
  { [ help: true ], _, _ }
    -> :help
  { _, [ user, project, count ], _ }
    -> { user, project, count }
  { _, [ user, project ], _ }
    -> { user, project, @default_count }
  _ -> :help
  { _, [ user, project ], _ } -> { user, project, @default_count }
  { _, [ _, project ], _ } -> { _, project, @default_count }
end

@tonini tonini force-pushed the correct-block-match-indentation branch from f3111ab to 0330d49 Compare August 4, 2015 17:57
tonini added a commit that referenced this pull request Aug 4, 2015
correct indentation for block with multiple matches
@tonini tonini merged commit f30fa7b into master Aug 4, 2015
@tonini tonini deleted the correct-block-match-indentation branch August 4, 2015 19:51
@tonini tonini mentioned this pull request Aug 4, 2015
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.

Indentation issues
1 participant