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

Pipe operator inside quotation expression leads to wrong indentation #256

Closed
danyx23 opened this issue May 13, 2018 · 3 comments
Closed

Comments

@danyx23
Copy link
Contributor

danyx23 commented May 13, 2018

The following expression:

test <@ results.GetAllResults() |> set = expected_outcome @>

Gets reformatted to this, which fails to compile (the equals sign should be aligned with set):

test <@ results.GetAllResults()
        |> set
        = expected_outcome @>

Does it make more sense to report these things in the form of failing unit tests? If so, this here fails:

[<Test>]
let ``Quotation expression with pipe and comparision``() =
    formatSourceString false """
    test <@ results.GetAllResults() |> set = expected_outcome @>""" config
    |> prepend newline
    |> should equal """
test <@ results.GetAllResults()
        |> set
           = expected_outcome @>
"""
@danyx23
Copy link
Contributor Author

danyx23 commented May 13, 2018

Might be a duplicate of #248

@nojaf
Copy link
Contributor

nojaf commented May 13, 2018

Failing unit test is the best way to report this. Interesting case.

@jindraivanek
Copy link
Contributor

Fixed by #294.

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

3 participants