Styleguide: Difference between revisions

From diaspora* project wiki
(Mention pronto)
(Add js styleguide)
Line 26: Line 26:
= JavaScript =
= JavaScript =


{{Work in progress}}
We use [http://jshint.com/docs/options/ JSHint] with our own [https://github.com/diaspora/diaspora/blob/develop/config/.jshint.json config]. Some of the most important rules are:
 
* All variable names have to use either camelCase style or UPPER_CASE with underscores.
* Always put curly braces around blocks in loops and conditionals.
* Use <tt>===</tt> and <tt>!==</tt> in favor of <tt>==</tt> and <tt>!=</tt>.
* Use two spaces per indentation level.
* Maximum line length is 120 characters.
* Use double quotes (<tt>"</tt>)


To be decided, please see https://www.loomio.org/d/bLH78pEh/styleguides


= SCSS =
= SCSS =

Revision as of 19:05, 28 August 2015

Please adhere to the following styleguides for your contributions. Transform existing code you touch to follow them.

Ruby

We follow bbatsov's styleguide for Ruby, with the following derivations and choices:

  • Maximum line length is 120 characters.
  • Always use raise, never use fail. Use abort to exit on a fatal error condition.
  • No assignment in conditions, not even so called (safe = assignment).
  • No enforced variable names for inject, use names that properly describe the data.
  • Do not use Class.new to define exceptions, use the regular class keyword.
  • Do not add spaces inside string interpolation, "a #{b} c" is valid, "a #{ b } c" is not.
  • Do not do control flow with && and || outside conditionals. The only valid uses of && and || are in a condition and to compute a predicate that’s returned from a method.
  • Do not add spaces around = when used to define default arguments in method definitions. def foo(a, b=c) is valid, def foo(a, b = c) is not.
  • Default to using double quotes ("), only use single quotes (') when you want to use a double quote in your string. Use a percent literal if you want to use both in the string.
  • Use the %i percent literal to define an array of symbols.
  • Prefer Hash#has_key? and Hash#has_value? over Hash#key? and Hash#value?.
  • Prefer String#% over Kernel#sprintf.
  • Prefer inject over reduce.
  • Do not put put a space between the opening brace and a block argument. foo {|a| } is valid, foo { |a| } is not.
  • Do not put a space after the opening brace or before the closing brace of a hash literal. {foo: bar} is valid, { foo: bar } is not.
  • Use Weirichs rule when deciding about about the block syntax to use.
  • Use an appropriate name for the argument when defining an operator method, don't just default to other.


JavaScript

We use JSHint with our own config. Some of the most important rules are:

  • All variable names have to use either camelCase style or UPPER_CASE with underscores.
  • Always put curly braces around blocks in loops and conditionals.
  • Use === and !== in favor of == and !=.
  • Use two spaces per indentation level.
  • Maximum line length is 120 characters.
  • Use double quotes (")


SCSS

Work in progressWork in progress:This article is a work in progress, it may contain incomplete or inaccurate information. There may be additional information on the talk page.

To be decided, please see https://www.loomio.org/d/bLH78pEh/styleguides


Automatic local review

You can quickly check your feature branch for introduced violations with $ bin/pronto run -c upstream/develop