(CPR-57) Style fixes

Now that the packaging repo has rubocop we need to have a baseline for
all future runs. This commit goes through and fixes all 316 detected
offenses.
IMO we use "fail" on purpose because it's more expressive. I'd prefer to keep it.