-
-
Notifications
You must be signed in to change notification settings - Fork 190
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
Removal of bar in one-case DU #1563
Comments
I've actually already identified another case where the pipe is always necessary (See #1483). |
(Personally I always use the pipe whether or not it's required.) |
Is this reported upstream? Disregarding formatting tooling, the issue is worth bringing to attention on the compiler repository. |
auduchinok beat me to it and raised dotnet/fsharp#11488. |
@Smaug123 I've been playing with xml doc edge cases while working on our tree, yeah. 🙂 I actually thought I've seen this issue being reported somewhere but couldn't find where it was! |
Issue created from fantomas-online
Code
Result
Problem description
The code Fantomas produces here is fine, and it compiles. However, it triggers what I assume is a compiler bug: the output documentation
.xml
file does not contain an entry for the DU caseBar
. This breaks our internal tests asserting that our API surfaces are documented.Would it be out of the question to give us the ability to always have the
| Bar of int
formatting?Extra information
Options
Fantomas Master at 03/31/2021 17:11:59 - 44926de
Signature file
Did you know that you can ignore files when formatting from fantomas-tool or the FAKE targets by using a .fantomasignore file?
The text was updated successfully, but these errors were encountered: