Replies: 1 comment
-
@delta1186 a PR addressing this would be awesome! This has bugged me for ages, it happens with nested arrays as well. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hey @loganhenson currently when you run tlint on a laravel project the snip below will throw the following lint issue: ! Multiline arrays should have trailing commas
Tlint is wanting ],]) but when I add this my php-cs-fixer file removes the comma. The only way to keep php-cs-fixer from removing the comma is to write is like this:
},
])
Of course this extra line looks awful. So in the situation shown above should tlint not throw this exception for this type of multiline array in laravel? I can work on a PR if you think the tlint exception is being thrown by mistake.
Beta Was this translation helpful? Give feedback.
All reactions