The Infinum Backend Team writes code following the community guidelines:
Please read them.
We use a Ruby style-checking analyzer called Rubocop to preserve syntax consistency in our applications. It's packaged as a Ruby gem and comes with a CLI to check style consistency. We run the checks through an Overcommit hook on almost all our projects (if it's missing on a legacy project, check with your colleagues whether it makes sense to install it).
We also use it as a plugin in our editors to get real time in-editor warnings just as we're writing code. Whether you're using Vim, Atom or Sublime, please check that it's properly installed after you run our installation scripts.
Rubocop is highly configurable, and we deviate only slightly from the suggested guidelines. Our configuration can be found here. Each project must have its own copy of rubocop.yml where project specific configuration can be added.
Some exceptions from our config file:
We avoid documenting every class or module. It doesn't make sense to write about a User, Post, or a Comment class and all of its methods. However, applications tend to have a Domain-Specific Business Logic that requires a documentation since, without a few comments, it's often not clear what's going on. If that's the case, please explain the interactions in the project README. If you're adding a hack or some method is implemented in a non-standard way, inline comment is mandatory.
The community guidelines suggest to write arrays with string elements with the percent notation:
developers = %w(Lucas John Tommy)
developers = ['Lucas', 'John', 'Tommy']
We find it OK to write string arrays the second way for two reasons:
- We never remember to write them with the percent notation in the first place, so we always have to change them afterwards.
- The percent notation looks odd when you have multiple words in one string:
developers = %w(Lucas John Tommy Herman\ Zvonimir)