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

Documentation contains reference to infix multiline formatter even though it's removed #1884

Closed
panmona opened this issue Sep 8, 2021 · 1 comment · Fixed by #1885
Closed

Comments

@panmona
Copy link
Contributor

panmona commented Sep 8, 2021

In the documentation under the description to fsharp_max_infix_operator_expression the property fsharp_infix_operator_expression_multiline_formatter is mentioned. This property no longer exists though.

That option sounds really interesting to us, so I would be interested into the reasons behind the change. I couldn't find any info in the respective PR (#1232).

But as I'm sure that there were valid reasons for this change, I suggest removing the note:

Requires fsharp_infix_operator_expression_multiline_formatter to be character_width to take effect.
@nojaf
Copy link
Contributor

nojaf commented Sep 8, 2021

Hello, you can read up on the thread at #1143.
In summary, this was something that I didn't want to pursue as it does not move the project forward.
The docs should be updated indeed. Feel free to submit a PR for this.

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