Skip to content

Commit

Permalink
Merge pull request #116 from github/francisfuzz/styleguide-syntax-update
Browse files Browse the repository at this point in the history
fix(STYLEGUIDE.md): use list item instead of codeblock
  • Loading branch information
bensheldon authored Oct 13, 2022
2 parents 2eddce7 + 2ab6fce commit 3f6ff03
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions STYLEGUIDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -797,9 +797,9 @@ names.select do |name|
end.map { |name| name.upcase }
```

Some will argue that multiline chaining would look OK with the use of {...}, but they should
ask themselves - is this code really readable and can't the block's contents be extracted into
nifty methods?
* Some will argue that multiline chaining would look OK with the use of `{...}`,
but they should ask themselves: is this code really readable and can't the block's
contents be extracted into nifty methods?

* Avoid `return` where not required.
<a name="avoid-return"></a><sup>[[link](#avoid-return)]</sup>
Expand Down

0 comments on commit 3f6ff03

Please sign in to comment.