Regression: An empty line or comment at the end of a list breaks Stroustrup formatting

Linschlager commented 2 months ago

Code

let array = [|
    someItem

|]

let array2 = [|
    someItem
    // comment
|]

Result

let array =
    [|
        someItem

    |]

let array2 =
    [|
        someItem
    // comment
    |]

Problem description

Placing an empty line or comment on the last line of an arrow, list or similar breaks Stroustrup formatting

Extra information

Might be related to one of the following:

[ ] The formatted result breaks my code.
[ ] The formatted result gives compiler warnings.
[ ] I or my company would be willing to help fix this.
[ ] I would like a release if this problem is solved.

Options

Fantomas main branch at 15/4/2024

    { config with
                MultilineBracketStyle = stroustrup }

_{Did you know that you can ignore files when formatting by using a .fantomasignore file?} _{PS: It's unlikely that someone else will solve your specific issue, as it's something that you have a personal stake in.}

nojaf commented 2 months ago

Hi, thank you for this report. This may have been introduced by https://github.com/fsprojects/fantomas/pull/3071. Are you interested in submitting a PR for this?

Linschlager commented 2 months ago

@nojaf Here you go https://github.com/fsprojects/fantomas/pull/3080

fsprojects / fantomas