Skip to content
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

Crash regression on 4.4.0-beta-003 #1438

Closed
TobyShaw opened this issue Feb 9, 2021 · 0 comments · Fixed by #1439
Closed

Crash regression on 4.4.0-beta-003 #1438

TobyShaw opened this issue Feb 9, 2021 · 0 comments · Fixed by #1439

Comments

@TobyShaw
Copy link

TobyShaw commented Feb 9, 2021

The online fantomas-tool gave a URL which was too long.
This also crashes on HEAD

For the following snippet:

namespace AspNetSerilog

[<Extension>]
type IWebHostBuilderExtensions() =

    [<Extension>]
    static member UseSerilog(webHostBuilder : IWebHostBuilder, index : Index) =
        webHostBuilder.UseSerilog(fun context configuration ->
            configuration
                .MinimumLevel.Debug()
                .WriteTo.Logger(fun loggerConfiguration ->
                    loggerConfiguration
                        .Enrich.WithProperty("host", Environment.MachineName)
                        .Enrich.WithProperty("user", Environment.UserName)
                        .Enrich.WithProperty("application", context.HostingEnvironment.ApplicationName)
                    |> ignore
                )
            |> ignore
        )

and the following .editorconfig:

[*.{fs,fsi}]
fsharp_space_before_uppercase_invocation=true
fsharp_space_before_class_constructor=true
fsharp_space_before_member=true
fsharp_space_before_colon=true
fsharp_space_before_semicolon=true
fsharp_indent_on_try_with=true
fsharp_multiline_block_brackets_on_same_column=true
fsharp_newline_between_type_definition_and_members=true
fsharp_align_function_signature_to_indentation=true
fsharp_alternative_long_member_definitions=true
fsharp_disable_elmish_syntax=true
fsharp_multi_line_lambda_closing_newline=true
max_line_length=100

It crashes

nojaf added a commit that referenced this issue Feb 9, 2021
* Only add a space at the end of a long nested DotGet chain. Fixes #1438. Fixes #1437.

* Bump to 4.4.0-beta-005.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants