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

Detect and enforce comma placement style #276

Closed
igaray opened this issue Sep 22, 2015 · 5 comments
Closed

Detect and enforce comma placement style #276

igaray opened this issue Sep 22, 2015 · 5 comments
Labels

Comments

@igaray
Copy link
Member

igaray commented Sep 22, 2015

e.g. Trailing commas vs leading commas

{a, 
  [ 1, 
    2, 
    3
  ]
}.

vs

{a
, [ 1 
  , 2 
  , 3
  ]
}.
@jfacorro jfacorro added the rule label Sep 22, 2015
@elbrujohalcon
Copy link
Member

Just a warning: you need to distinguish between commas in lists and tuples and commas separating expressions.
I'm pretty sure that will not be trivial.

@paulo-ferraz-oliveira
Copy link
Collaborator

I think rebar3_format addresses this issue, kind of, doesn't it?

@elbrujohalcon
Copy link
Member

It does, indeed.

@jfacorro
Copy link
Contributor

🎉 rebar3_format already supports comma first?

@elbrujohalcon
Copy link
Member

No. It just supports consistent comma placement :trollface:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants