-
Notifications
You must be signed in to change notification settings - Fork 15
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
AA style #30
Comments
I think the options you have for trailing comas is good. Keep in mind this can also apply for arrays so a more general rule for trailing comas might be good. Comas are required for single line or same line values. Again this also applies to arrays. For quotes I can still see value in a rule for it. I would think by default the rule would be no enforcement. And if you want a
I have had this as a rule for many projects. |
this seems right to me. |
@chrisdp ok so AA keys options to optionally enforce quote / no-quote (and obviously |
@elsassph Yes that sounds right. |
Reopning because we still want AA key quotes rules. |
What options do we want for AA literals?
What about commas? Commas are optional - I'm thinking options could be:
Does that work both for multi-line/one-line literals?
Other things related to linting? I don't think using quotes should be enforced as it changes the semantics of the AA.
The text was updated successfully, but these errors were encountered: