diff options
| author | Christian Neukirchen <chneukirchen@gmail.com> | 2010-04-09 21:57:52 +0200 |
|---|---|---|
| committer | Christian Neukirchen <chneukirchen@gmail.com> | 2010-04-09 21:57:52 +0200 |
| commit | 436e8c1bbeb6ffde8bcfbcf72af449c5f8a741f3 (patch) | |
| tree | e27bab93144ddf3aab528cf197c4a7388a2f1b1b | |
| parent | 5cb39d0935df6678ee0ab9036f5dd2056217e9a6 (diff) | |
| download | styleguide-436e8c1bbeb6ffde8bcfbcf72af449c5f8a741f3.tar.gz styleguide-436e8c1bbeb6ffde8bcfbcf72af449c5f8a741f3.tar.xz styleguide-436e8c1bbeb6ffde8bcfbcf72af449c5f8a741f3.zip | |
Small additions based on what I saw in the forks
| -rw-r--r-- | RUBY-STYLE | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/RUBY-STYLE b/RUBY-STYLE index 5773609..eb10450 100644 --- a/RUBY-STYLE +++ b/RUBY-STYLE @@ -118,7 +118,8 @@ when you contribute to my code, please follow these rules: * When defining binary operators, name the argument "other". -* Prefer map over collect, find over detect, find_all over select. +* Prefer map over collect, find over detect, find_all over select, + size over length. == Comments: @@ -157,6 +158,8 @@ when you contribute to my code, please follow these rules: * Code in a functional way, avoid mutation when it makes sense. +* Do not mutate arguments unless that is the purpose of the method. + * Do not mess around in core classes when writing libraries. * Do not program defensively. |