Merge pull request #244 from ifad/chore/use-guard-clauses

Fix Style/GuardClause offenses

lib/active_record/connection_adapters/chronomodel_adapter.rb

@@ -41,11 +41,9 @@ def chronomodel_connection(config) # :nodoc:
 
       adapter
     rescue ::PG::Error => e
-      if e.message.include?(conn_params[:dbname])
-        raise ActiveRecord::NoDatabaseError
-      else
-        raise
-      end
+      raise ActiveRecord::NoDatabaseError if e.message.include?(conn_params[:dbname])
+
+      raise
     end
   end
 end

lib/chrono_model/adapter/migrations.rb

@@ -201,9 +201,9 @@ def chrono_make_temporal_table(table_name, options)
         # Optionally copy the plain table data, setting up history
         # retroactively.
         #
-        if options[:copy_data]
-          chrono_copy_temporal_to_history(table_name, options)
-        end
+        return unless options[:copy_data]
+
+        chrono_copy_temporal_to_history(table_name, options)
       end
 
       def chrono_copy_temporal_to_history(table_name, options)

lib/chrono_model/time_machine.rb

@@ -74,9 +74,9 @@ def inherited(subclass)
           # Sadly, we can't avoid it by calling +.history?+, because in the
           # subclass the HistoryModel hasn't been included yet.
           #
-          unless subclass.name.nil?
-            ChronoModel::TimeMachine.define_history_model_for(subclass)
-          end
+          return if subclass.name.nil?
+
+          ChronoModel::TimeMachine.define_history_model_for(subclass)
         end
       end
     end