Comment inside LongIdentWithDots not preserved

[dbrattli]

Issue created from fantomas-online

Code

let path =
    match normalizedPath with
    | path ->
        path  // translate path to Python relative syntax
            .Replace("../../../", "....")
            

Result

let path =
    match normalizedPath with
    | path -> path.Replace("../../../", "....")

Problem description

Fantomas removes comment in method dot chain. It looks tricky since the white-space where the
comment was has been removed. Perhaps the comment should be moved after the .Replace instead? Related to fable-compiler/Fable#2738

Extra information

• The formatted result breaks by code.
• The formatted result gives compiler warnings.
• I or my company would be willing to help fix this.

Options

Fantomas master branch at 1/1/1990

Default Fantomas configuration

_{Did you know that you can ignore files when formatting from fantomas-tool or the FAKE targets by using a .fantomasignore file?}

[nojaf]

Hello Dag, thank you for reporting this issue.
Fantomas is not considering trivia right now inside a LongIdentWithDots right now.

LongIdentWithDots([path; Replace], [tmp.fsx (5,12--5,13)])

Comments between dots are not printed, an easy workaround might be to move the comment above or below it.

Are you interested in submitting a PR for this one?

[dbrattli]

Hi Florian, thanks for the info. Yes I can try. It would be interesting to learn more about the untyped AST. I'll have a look at it (next weekend)

[nojaf]

Cool to hear! To get started, you can check out our Contribution Guidelines and take a peek at the YouTube channel.
If you have any questions, feel free to reach out.

[dbrattli]

@nojaf Sorry that I didn't have the bandwidth for looking into this

[nojaf]

No worries, this would have been quite the challenge anyway for someone new to the codebase to pick up.
That being said, I'm working on other aspects of this project. So to see your own issues moving, I highly recommend scratching your own itch 😊.
I'm going to release this shortly as part of 4.6.2.