enforce double quote style and run rubocop on the project with --safe-auto-correct

Author: tarzan

.rubocop.yaml .rubocop.yml.rubocop.yaml renamed to .rubocop.yml

@@ -1,7 +1,7 @@
 Style/StringLiterals:
   Description: Checks if uses of quotes match the configured preference.
   StyleGuide: https://github.com/bbatsov/ruby-style-guide#consistent-string-literals
-  Enabled: false
+  Enabled: true
   EnforcedStyle: double_quotes
   SupportedStyles:
   - single_quotes

Gemfile

@@ -3,54 +3,54 @@ source "https://rubygems.org"
 ruby "2.4.2"
 
 gem "airbrake"
+gem "apartment"
+gem "audited"
+gem "aws-sdk"
+gem "bitters"
 gem "bourbon"
+gem "brakeman"
 gem "coffee-rails"
 gem "delayed_job_active_record"
+gem "devise", "~> 4.7.1"
+gem "devise_invitable"
 gem "email_validator"
 gem "flutie"
+gem "gravatar_image_tag"
+gem "haml-rails"
+gem "hashtel", "~> 0.0.2"
 gem "high_voltage"
+gem "holidays"
+gem "http_accept_language"
+gem "jquery-atwho-rails", "~> 1.3.2" # autocomplete
 gem "jquery-rails", "~> 4.0"
+gem "kaminari"
+gem "momentjs-rails"
 gem "neat"
-gem "bitters"
+gem "normalize-rails"
+gem "paperclip", "~> 6.1.0"
 gem "pg"
+gem "pikaday-gem", "~> 1.2.0.0"
 gem "rack-timeout"
 gem "rails", "~> 5.0.0"
 gem "recipient_interceptor"
+gem "redcarpet"
 gem "sass-rails", "~> 5.0.1"
+gem "select2-rails"
 gem "simple_form"
+gem "sprockets-rails", "~>3.0"
 gem "title"
+gem "twitter-text" # hashtag parsing
 gem "uglifier"
 gem "unicorn"
-gem "devise", "~> 4.7.1"
-gem "devise_invitable"
-gem "apartment"
-gem "pikaday-gem", "~> 1.2.0.0"
-gem "momentjs-rails"
-gem "gravatar_image_tag"
-gem "hashtel", "~> 0.0.2"
-gem "kaminari"
-gem "select2-rails"
-gem "http_accept_language"
-gem "normalize-rails"
-gem "twitter-text" # hashtag parsing
-gem "jquery-atwho-rails", "~> 1.3.2" # autocomplete
-gem "haml-rails"
-gem "audited"
-gem "paperclip", "~> 6.1.0"
-gem "aws-sdk"
-gem "redcarpet"
-gem "holidays"
-gem "sprockets-rails", "~>3.0"
-gem "brakeman"
 
 source "https://rails-assets.org" do
   gem "rails-assets-chartjs"
 end
 
 # caching
 
-gem "kgio" # faster I/O
 gem "dalli" # memcached
+gem "kgio" # faster I/O
 gem "memcachier"
 
 group :development do
@@ -63,17 +63,17 @@ group :development do
 end
 
 group :development, :test do
+  gem "annotate"
   gem "dotenv-rails"
+  gem "email_spec"
   gem "factory_girl_rails"
+  gem "letter_opener"
   gem "pry-rails"
   gem "rspec-rails", "~> 3.8"
-  gem "annotate"
-  gem "letter_opener"
-  gem "email_spec"
 end
 
 group :test do
-  gem 'capybara'
+  gem "capybara"
   gem "database_cleaner"
   gem "launchy"
   gem "shoulda-matchers", "~> 2.7.0"

Rakefile

@@ -3,34 +3,34 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require File.expand_path('../config/application', __FILE__)
+require File.expand_path("config/application", __dir__)
 
 Hours::Application.load_tasks
 if defined?(RSpec)
   task(:spec).clear
 
-  desc 'Run all specs'
+  desc "Run all specs"
   RSpec::Core::RakeTask.new(:spec) do |t|
-    t.rspec_opts = '--tag ~factory'
+    t.rspec_opts = "--tag ~factory"
   end
 
-  desc 'Run factory specs.'
+  desc "Run factory specs."
   RSpec::Core::RakeTask.new(:factory_specs) do |t|
-    t.pattern = './spec/models/factories_spec.rb'
+    t.pattern = "./spec/models/factories_spec.rb"
   end
 
   task spec: :factory_specs
 end
 
 namespace :brakeman do
-  desc 'Run Brakeman'
+  desc "Run Brakeman"
   task :check, :output_files do |_t, args|
-    require 'brakeman'
+    require "brakeman"
 
-    files = args[:output_files].split(' ') if args[:output_files]
-    Brakeman.run app_path: '.', output_files: files, print_report: true
+    files = args[:output_files].split(" ") if args[:output_files]
+    Brakeman.run app_path: ".", output_files: files, print_report: true
   end
 end
 
 task(:default).clear
-task default: [:spec, 'brakeman:check']
+task default: [:spec, "brakeman:check"]

app/controllers/accounts_controller.rb

@@ -1,6 +1,6 @@
 class AccountsController < ApplicationController
-  skip_before_action :authenticate_user!, only: [:new, :create]
-  before_action :authorize!, only: [:edit, :destroy]
+  skip_before_action :authenticate_user!, only: %i[new create]
+  before_action :authorize!, only: %i[edit destroy]
 
   def new
     @signup = Signup.new
@@ -32,7 +32,7 @@ def destroy
   def signup_params
     params.require(:signup)
           .permit(:subdomain, :first_name, :last_name, :email,
-                    :password, :password_confirmation)
+                  :password, :password_confirmation)
   end
 
   def authorize!

app/controllers/application_controller.rb

@@ -44,7 +44,7 @@ def current_account
   end
 
   def load_schema
-    Apartment::Tenant.switch!('public')
+    Apartment::Tenant.switch!("public")
     return unless request.subdomain.present?
 
     if current_account

app/controllers/audits_controller.rb

@@ -8,13 +8,12 @@ def index
   private
 
   def audit_log
-    case
-    when params.key?(:hour_id)
-      return hour_log
-    when params.key?(:mileage_id)
-      return mileage_log
-    when params.key?(:project_id)
-      return project_log
+    if params.key?(:hour_id)
+      hour_log
+    elsif params.key?(:mileage_id)
+      mileage_log
+    elsif params.key?(:project_id)
+      project_log
     end
   end
 

app/controllers/categories_controller.rb

@@ -1,5 +1,5 @@
 class CategoriesController < ApplicationController
-  before_action :find_category, only: [:edit, :update]
+  before_action :find_category, only: %i[edit update]
 
   def index
     @category = Category.new
@@ -16,8 +16,7 @@ def create
     end
   end
 
-  def edit
-  end
+  def edit; end
 
   def update
     if @category.update(category_params)

app/controllers/entries_controller.rb

@@ -7,15 +7,17 @@ def index
     @user = User.find_by_slug(params[:user_id])
     @hours_entries = @user.hours.by_date.page(params[:hours_pages]).per(20)
     @mileages_entries = @user.mileages.by_date.page(
-      params[:mileages_pages]).per(20)
+      params[:mileages_pages]
+    ).per(20)
 
     respond_to do |format|
       format.html { @mileages_entries + @hours_entries }
       format.csv do
         send_csv(
           name: @user.name,
           hours_entries: @user.hours.by_date,
-          mileages_entries: @user.mileages.by_date)
+          mileages_entries: @user.mileages.by_date
+        )
       end
     end
   end

app/controllers/hours_controller.rb

@@ -30,8 +30,8 @@ def resource
   end
 
   def entry_params
-    params.require(:hour).
-      permit(:project_id, :category_id, :value, :description, :date).
-      merge(date: parsed_date(:hour))
+    params.require(:hour)
+          .permit(:project_id, :category_id, :value, :description, :date)
+          .merge(date: parsed_date(:hour))
   end
 end

app/controllers/mileages_controller.rb

@@ -33,8 +33,8 @@ def resource
   end
 
   def entry_params
-    params.require(:mileage).
-      permit(:project_id, :value, :date).
-      merge(date: parsed_date(:mileage))
+    params.require(:mileage)
+          .permit(:project_id, :value, :date)
+          .merge(date: parsed_date(:mileage))
   end
 end