Fix some formatting in linter README

One of the tables had an extra row, and one of the code examples did not
have any syntax highlighting. Fix both.

Change-Id: Ie653491af926de58d88edd31511f4c48d51444ba
Reviewed-on: http://gerrit.causes.com/36700
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Ignore single in blocks in EmptyLineBetweenBlocks

Add support for ignoring single line blocks in the
`EmptyLineBetweenBlocks` linter, and make it configurable via the
`ignore_single_line_blocks` boolean option.

Closes #92

Change-Id: I37aad18dbf4ecc6108d4de5dbfe34b6f32ccfc1d
Reviewed-on: http://gerrit.causes.com/36701
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Ignore ClassAndModuleChildren Rubocop cop

Enforcing this results in a lot of unnecessary indentation.

Change-Id: I53f8d5b561317c53c9ec1050a38a71085c8ea74c
Reviewed-on: http://gerrit.causes.com/36702
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Ignore Documentation Rubocop cop in specs

We create a lot of one-off classes in specs, and Rubocop was complaining
that these weren't documented. Ignore these offences.

Change-Id: I7da0b96f3810e228e3bc7c59216322c88424de92
Reviewed-on: http://gerrit.causes.com/36703
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Ignore FileName cop for scss-lint binary

Hyphens are better than underscores when it comes to command line tools.

Change-Id: I5151dd6bc6393aceb0eeea6cb558aad180082d0e
Reviewed-on: http://gerrit.causes.com/36704
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Customize PercentLiteralDelimiters cop

Square brackets emphasize that the %{i,w,W} calls return arrays.

Change-Id: Idf2f271e08f2c0ae72e9034d9762bcba1571d8f9
Reviewed-on: http://gerrit.causes.com/36705
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Ignore PredicateName Rubocop cop

This obfuscates whether `something?` is a "is-a" or a "has-a"
relationship.

Change-Id: Ibddb6ae0e071643452d9197bf8e23aa08d5c1fed
Reviewed-on: http://gerrit.causes.com/36706
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Ignore SingleLineBlockParams Rubocop cop

This prevents us from choosing intention-revealing names for blocks
passed to `inject` and the like.

Change-Id: I6b95a184dfe086ff0f0604f75a7960ec68807a22
Reviewed-on: http://gerrit.causes.com/36707
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix Rubocop lints in Engine class

Change-Id: Ib6135d18727a078277810f78ecfc516085e1362d
Reviewed-on: http://gerrit.causes.com/36708
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix Rubocop lints in ColorKeyword linter

Change-Id: I9015955323b37eb087829e01c45c80d8c8c6bb7e
Reviewed-on: http://gerrit.causes.com/36709
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix Rubocop lints in IdWithExtraneousSelector

Change-Id: Ib06958a78c715c01349fa38db793565fdfa771d9
Reviewed-on: http://gerrit.causes.com/36710
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Prefer alias_method over alias

Appease the Rubocop gods.

Change-Id: Icdc7568cc5dc79b6c0bc14a3f4e5c77db89f87a1
Reviewed-on: http://gerrit.causes.com/36711
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix indentation in BorderZero linter

Rubocop noticed a constant was misaligned.

Change-Id: Ie5296be458d51a4f1700b2df5a36fa7dd8478695
Reviewed-on: http://gerrit.causes.com/36712
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix Rubocop lints in StringQuotes linter

Change-Id: I34990fa7b91e1c019bca03a50ee6b8964ab2af81
Reviewed-on: http://gerrit.causes.com/36713
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix Rubocop lints in Config spec

Change-Id: I7eca3df0a365d3094a7d4f9f6dba6cefd42db7c4
Reviewed-on: http://gerrit.causes.com/36714
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Simplify complexity of condensed_shorthand

Rubocop was reporting that this method was two complex. Extract some
helper methods to make it more intention-revealing at the expense of
slightly more code.

Change-Id: I311a9261bc6184233adbe9856c331af55e8b5b39
Reviewed-on: http://gerrit.causes.com/36715
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix Rubocop lints in PropertySortOrder

Extract some helper functions from the complex `compare_properties`
method.

Also fix the multi-line method chain lint.

Change-Id: I0b01db044e362a39063b052eb9cb49e19467ef13
Reviewed-on: http://gerrit.causes.com/36716
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Add UrlFormat linter

Add linter which checks for URLs with protocols or domains. These should
be avoided as they make your CSS more brittle to change.

Change-Id: Ia9238f9c8e03ea326b3d2265c162070d393a564f
Reviewed-on: http://gerrit.causes.com/36717
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Don't define description method in BorderZero

We're going to deprecate the `description` method as it isn't as
customizable and makes the code a bit harder to read.

While removing the `description` method it was noticed that some
parentheses were missing and that the format of the border property
names could be improved.

Change-Id: I0848eff895ac13153678a80da718e300ff413e25
Reviewed-on: http://gerrit.causes.com/36718
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Remove description method from Comment linter

This was unnecessary and is being deprecated.

Change-Id: Ie53665206f3769152f61b78c2fef2ff78f3d9334
Reviewed-on: http://gerrit.causes.com/36720
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>