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

Multiple List concat operators are not seen as the same operator. #2472

Closed
3 tasks
nojaf opened this issue Sep 1, 2022 · 2 comments
Closed
3 tasks

Multiple List concat operators are not seen as the same operator. #2472

nojaf opened this issue Sep 1, 2022 · 2 comments

Comments

@nojaf
Copy link
Contributor

nojaf commented Sep 1, 2022

Issue created from fantomas-online

Code

        let allDecls = inheritsL @ iimplsLs @ ctorLs @ instanceValLs @ methLs @ ilFieldsL @ propLs @ eventLs @ staticValLs @ nestedTypeLs

        let allDecls = inheritsL + iimplsLs + ctorLs + instanceValLs + methLs + ilFieldsL + propLs + eventLs + staticValLs + nestedTypeLs

Result

let allDecls =
    inheritsL
    @ iimplsLs
      @ ctorLs
        @ instanceValLs
          @ methLs @ ilFieldsL @ propLs @ eventLs @ staticValLs @ nestedTypeLs

let allDecls =
    inheritsL
    + iimplsLs
    + ctorLs
    + instanceValLs
    + methLs
    + ilFieldsL
    + propLs
    + eventLs
    + staticValLs
    + nestedTypeLs

Problem description

Fantomas has heurstic that if the operators are all the same in a large nested infix application, it will place them all on a new line. For some reason, this isn't working for @.

Extra information

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

Options

Fantomas master branch at 2022-09-01T05:33:17Z - 582e9ab

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 nojaf added bug (stylistic) good first issue Long hanging fruit: easy issue to get your feet wet! labels Sep 1, 2022
@nojaf
Copy link
Contributor Author

nojaf commented Sep 1, 2022

The problem isn't limited to @, sample with ^^

@nojaf nojaf removed the good first issue Long hanging fruit: easy issue to get your feet wet! label Dec 26, 2022
@nojaf
Copy link
Contributor Author

nojaf commented Jan 2, 2023

Fix available in v5.2.0-beta-001

@nojaf nojaf closed this as completed Jan 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant