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

Computation expression placement #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 51 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ These guidelines are based on the official [F# code formatting guidelines](https
* [Formatting attributes on parameters](#formatting-attributes-on-parameters)
* [Formatting multiple attributes](#formatting-multiple-attributes)
* [Formatting literals](#formatting-literals)
* [Formatting computation expressions](#formatting-computation-expressions)

## General rules for indentation

Expand Down Expand Up @@ -1053,3 +1054,53 @@ let MyUrl = "www.mywebsitethatiamworkingwith.com"
```

Avoid placing the attribute on the same line as the value.

## Formatting computation expressions

[Computation expressions](https://docs.microsoft.com/en-us/dotnet/fsharp/language-reference/computation-expressions) should be formatted with the computation expression builder keyword on a new line, indented one place.

```fsharp
let items =
seq {
yield "item1"
yield "item2"
}

let items2 () =
seq {
yield "item1"
yield "item2"
}

let foo () : Async<int> =
async {
let! blah = Async.Sleep 100.
return 3
}

let bar () =
let foo () =
async {
let! blah = Async.Sleep 100.
return 3
}
4

let foo =
async {
let! blah = Async.Sleep 100.
return 3
}

let quux () =
let foo =
async {
let! blah = Async.Sleep 100.
let! baz =
async {
return 4
}
return 3
}
4
```