We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I propose we add configuration (or make default) for having consistent indentation of match cases. So:
The advantages of making this adjustment to Fantomas are - more consistent code look and feel.
The disadvantages of making this adjustment to Fantomas are - reformatting existing code base (could be fixed by disabling feature by default)
From Oxpecker Existing formatting:
match result with | Ok _ -> return! ctx.Write <| Created() | Error error -> env.Logger.LogError(error) return! ctx.Write <| BadRequest {| Error = error |}
Proposed formatting:
Estimated cost (XS, S, M, L, XL, XXL): M
Related suggestions: (put links to related suggestions here)
Please tick this by placing a cross in the box:
Please tick all that apply:
The text was updated successfully, but these errors were encountered:
This falls under what-are-we-not-looking-for.
You are in luck, however, that this was discussed at some point in the style guide: fsharp/fslang-design#650.
I would accept a PR for this.
Sorry, something went wrong.
No branches or pull requests
I propose we add configuration (or make default) for having consistent indentation of match cases.
So:
Pros and Cons
The advantages of making this adjustment to Fantomas are - more consistent code look and feel.
The disadvantages of making this adjustment to Fantomas are - reformatting existing code base (could be fixed by disabling feature by default)
Examples
From Oxpecker
Existing formatting:
Proposed formatting:
Extra information
Estimated cost (XS, S, M, L, XL, XXL): M
Related suggestions: (put links to related suggestions here)
Affidavit (please submit!)
Please tick this by placing a cross in the box:
Please tick all that apply:
The text was updated successfully, but these errors were encountered: