-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
// csharpier-ignore #581
Comments
@dlech Can you list the scenarios you were having issues with? Maybe we can use that to improve csharpier itself. One is tabular arrays and functions arguments that the prettier example gives. |
Yes, arrays like this: https://github.com/dlech/SshAgentLib/blob/07c4d17f6648d13378dee274ebc8efa3c9b78144/SshAgentLib/Crypto/BCrypt.cs#L355 It makes for way too much scrolling when each item is on a new line. |
I like what google-java-format does. They have an exception for tabular arrays to keep the original formatting. This is in line with the no-options philosophy. Although if we go this route, this might not be the only exception we'd need to add. So much more work. |
I think it makes sense to support I created #583 because even with support for ignoring lines, I think it may make sense to leave tabular initializers alone. |
0.15.0 is out now with the support for |
There are some cases where initializers have super-long lists of arguments. It would be helpful to have the equivalent of // prettier-ignore to handle these cases.
The text was updated successfully, but these errors were encountered: