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

Relax indentation rules on anonymous records #786

Closed
5 tasks done
Happypig375 opened this issue Sep 21, 2019 · 2 comments
Closed
5 tasks done

Relax indentation rules on anonymous records #786

Happypig375 opened this issue Sep 21, 2019 · 2 comments

Comments

@Happypig375
Copy link
Contributor

Relax indentation rules on anonymous records

I propose we relax indentation rules on anonymous records.

The existing way of approaching this problem in F# is indenting code blocks further, leaving wasted space.

let f x = x
let a = f [
    2 // List: OK
]
let b = [|
    2 // Array: OK
|]
type X = { X : int }
let c = f {
    X = 2 // Record: OK
}
let d = f {|
    X = 2 (* FS0058	Possible incorrect indentation:
this token is offside of context started at position (12:11).
Try indenting this token further or using standard formatting conventions. *)
|}
let e = f {|
            X = 2 // Indenting further is needed
        |}

Pros and Cons

The advantages of making this adjustment to F# are

  1. Parity
  2. Convenience
  3. Effective use of space
  4. Clarity

The disadvantages of making this adjustment to F# are none.

Extra information

Estimated cost (XS, S, M, L, XL, XXL): S

Related suggestions: (put links to related suggestions here)

Affidavit (please submit!)

Please tick this by placing a cross in the box:

  • This is not a question (e.g. like one you might ask on stackoverflow) and I have searched stackoverflow for discussions of this issue
  • I have searched both open and closed suggestions on this site and believe this is not a duplicate
  • This is not something which has obviously "already been decided" in previous versions of F#. If you're questioning a fundamental design decision that has obviously already been taken (e.g. "Make F# untyped") then please don't submit it.

Please tick all that apply:

  • This is not a breaking change to the F# language design
  • I or my company would be willing to help implement and/or test this
@cartermp
Copy link
Member

I agree with this since we've made it a precedent to relax indentation like this in the past few releases.

@dsyme
Copy link
Collaborator

dsyme commented Aug 24, 2021

Completed in preview dotnet/fsharp#11772

@dsyme dsyme closed this as completed Mar 19, 2022
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