FsAutoComplete icon indicating copy to clipboard operation
FsAutoComplete copied to clipboard

Add a code fix for adding missing `seq` before `{…}`

Open edgarfgp opened this issue 1 year ago • 0 comments

Details

I propose we add a code fix for adding missing seq before {…}. This is related to a language suggestion [Deprecate { start..finish } and { start..step..finish }] (https://github.com/fsharp/fslang-suggestions/issues/1033) implemented in https://github.com/dotnet/fsharp/pull/17772 which is currently in preview

  • Similar to the VS https://github.com/dotnet/fsharp/pull/17772/commits/07955068028bd2412d5e174fceb061e8fb2ec75a

Before

let xs = { 1..10 }

let xs = { 1; 10 }

let xs = id { 1..10 }

let xs = id { 1; 10 }

let xs =
    id {
        1..10
    }

let xs =
    id {
        1; 10
    }

After

let xs = seq { 1..10 }

let xs = seq { 1; 10 }

let xs = id (seq { 1..10 })

let xs = id (seq { 1; 10 })

let xs =
    id (seq {
        1..10
    })

let xs =
    id (seq {
        1; 10
    })

Checklist

  • [x] I have looked through existing issues to make sure that this feature has not been requested before
  • [x] I have provided a descriptive title for this issue
  • [x] I am aware that even valid feature requests may be rejected if they do not align with the project's goals
  • [x] I or my company would be willing to contribute this feature

edgarfgp avatar Dec 18 '24 17:12 edgarfgp