Skip to content

Commit

Permalink
Disallow the pipe operator (#15)
Browse files Browse the repository at this point in the history
Co-Authored-By: Jarrett Revels <jarrettrevels@gmail.com>
  • Loading branch information
ararslan and jrevels committed Nov 11, 2019
1 parent 4ab09b5 commit ed0ddf9
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ Note that lines containing only comments are not considered empty lines.
- If a field type annotation in a struct definition would default to `Any`, explicitly annotate the field with `::Any`.
- In argument lists, always separate positional arguments from keyword arguments using a semicolon (`;`).
- In method signatures, only provide names to arguments that are actually referenced in the method body (e.g. `f(x::Number, y) = y` is bad, `f(::Number, y) = y` is good).
- Do not use the "pipe" operator (`|>`). The primary use case for `|>` is to break up nested function calls. For this scenario, one should instead favor assigning intermediary results to well-named variables. This makes code easier to read, debug, and piecewise evaluate in the REPL. Another reason to avoid `|>` is that it clutters stacktraces.
## Programming Guidelines
Expand Down

0 comments on commit ed0ddf9

Please sign in to comment.