Fix NameFormat documentation: change `-` to `_`

As pointed out by @roadhump, the default value for the `convention`
option is `hyphenated_lowercase`, not the `hyphenated-lowercase` that
appeared in the documentation.

Fixes #147

Change-Id: I60d86e728dbff06997e47c909629948bd69743af
Reviewed-on: http://gerrit.causes.com/37554
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Joe Lencioni <joe.lencioni@causes.com>
Pin Rubocop version to 0.20.1

The most recent version of Rubocop adds some lints which results in our
build failing.

Since we can't predict when Rubocop will be updated, it is best to pin
the version so we can control whether or not the build breaks.

Change-Id: I585af6f01ab2d344e3802fea762c0e812e4246e8
Reviewed-on: http://gerrit.causes.com/37557
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Extend DuplicateRoot to MergeableSelector

Extends the behavior to check nested rule sets in addition to root-level
ones.

Change-Id: I251dd13ef6d9d2a3ef504be3e699ead5ce81b929
Reviewed-on: http://gerrit.causes.com/37604
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Clean up MergeableSelector docs and style

Some of the wording for the documentation was a bit off. Fix that and
some style in code and specs.

Change-Id: I7a21bf73e5c40ba35736582227bdf4d80b4aa75a
Reviewed-on: http://gerrit.causes.com/37605
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Change lint description for MergeableSelector

The grammar for this message was confusing and not helpful. Simplify it
to simply refer to the line of the rule the linter wants you to merge
the duplicate/nestable rule with.

Change-Id: I17578d72714af43d44831dac2b560181a467a79e
Reviewed-on: http://gerrit.causes.com/37606
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Enable RegexpLiteral cop

Change-Id: Ifd25e149edf96379bfa406f8c60bcf8939762bc3
Reviewed-on: http://gerrit.causes.com/37607
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Enable Void cop

Change-Id: I5a201f8ce5416bf9b6c09a21302138102f8ad735
Reviewed-on: http://gerrit.causes.com/37608
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Enable SpecialGlobalVars cop

Change-Id: I98c2f4d4f443f005c9e76660a7f2978c5d3eb4a4
Reviewed-on: http://gerrit.causes.com/37609
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Enable AmbiguousRegexpLiteral cop

Change-Id: If27e20ad415e76b64ef47071606602b16bf64bcf
Reviewed-on: http://gerrit.causes.com/37610
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Enable GuardClause cop

Change-Id: I03f6b165e11000db607fc9e1d42048f94887e878
Reviewed-on: http://gerrit.causes.com/37611
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Fix some GuardClause Rubocop lints

Change-Id: I4a0a1184b93f964d0fa362e22d6de138b24330d8
Reviewed-on: http://gerrit.causes.com/37612
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Update Rubocop to 0.21.0

Change-Id: I5cf9c49962767ff816dc915d6bc056f980040896
Reviewed-on: http://gerrit.causes.com/37613
Reviewed-by: Shane da Silva <shane@causes.com>
Tested-by: Shane da Silva <shane@causes.com>
Use next instead of if at end of iteration

There is a new cop coming called `Next` that will look for this case.

Change-Id: I35ac6acd33065aa4a0b7b7ce6caea711a80b6670
Reviewed-on: http://gerrit.causes.com/37614
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Shane da Silva <shane@causes.com>
Use SVG Badges

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