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
Consider
type DU = MyCase of (string | // but why? null)
Without having the range of |, we can't link the comment to the correct syntax node in Fantomas. Would be great if we could have the range in the AST.
|
https://fsharp.github.io/fsharp-compiler-docs/reference/fsharp-compiler-syntax-syntype.html#WithNull
null
not null
Windows (Default)
.NET SDK (.NET Core, .NET 5+)
FCS 43.9.100-preview.24422.2
No response
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Issue description
Consider
Without having the range of
|
, we can't link the comment to the correct syntax node in Fantomas.Would be great if we could have the range in the AST.
https://fsharp.github.io/fsharp-compiler-docs/reference/fsharp-compiler-syntax-syntype.html#WithNull
Choose one or more from the following categories of impact
null
constructs in code not using the checknulls switch.null
,not null
).Operating System
Windows (Default)
What .NET runtime/SDK kind are you seeing the issue on
.NET SDK (.NET Core, .NET 5+)
.NET Runtime/SDK version
FCS 43.9.100-preview.24422.2
Reproducible code snippet and actual behavior
Possible workarounds
No response
The text was updated successfully, but these errors were encountered: