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

Insertion of space before function application can break dot-chaining #943

Closed
1 of 3 tasks
Smaug123 opened this issue Jul 2, 2020 · 2 comments · Fixed by #962
Closed
1 of 3 tasks

Insertion of space before function application can break dot-chaining #943

Smaug123 opened this issue Jul 2, 2020 · 2 comments · Fixed by #962

Comments

@Smaug123
Copy link
Contributor

Smaug123 commented Jul 2, 2020

Issue created from fantomas-online

Code

foo.Bar().[5]

Result

foo.Bar ().[5]

Problem description

The space was necessary so that we could continue dotting into the expression. Looks very similar to #566.

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 at 06/30/2020 07:24:01 - dcc70c3

Name Value
IndentSpaceNum 4
PageWidth 120
SemicolonAtEndOfLine false
SpaceBeforeParameter true
SpaceBeforeLowercaseInvocation true
SpaceBeforeUppercaseInvocation true
SpaceBeforeClassConstructor true
SpaceBeforeMember true
SpaceBeforeColon true
SpaceAfterComma true
SpaceBeforeSemicolon true
SpaceAfterSemicolon true
IndentOnTryWith false
SpaceAroundDelimiter true
MaxIfThenElseShortWidth 40
MaxInfixOperatorExpression 50
MaxRecordWidth 40
MaxArrayOrListWidth 40
MaxValueBindingWidth 40
MaxFunctionBindingWidth 40
MultilineBlockBracketsOnSameColumn true
NewlineBetweenTypeDefinitionAndMembers true
KeepIfThenInSameLine true
MaxElmishWidth 40
SingleArgumentWebMode false
StrictMode false
@knocte
Copy link
Contributor

knocte commented Jul 3, 2020

I think this is a duplicate of #853

@Smaug123
Copy link
Contributor Author

Smaug123 commented Jul 3, 2020

Yep, thanks for finding the dupe!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants