Merge pull request #330 from Dynamoid/minor-refactoring-and-cleanup

Minor refactoring and cleanup

Author: andrykonchin

.rubocop.yml

@@ -8,12 +8,28 @@ AllCops:
 # It's a matter of taste
 Layout/AlignParameters:
   EnforcedStyle: with_fixed_indentation
+Layout/AlignHash:
+  Enabled: false
 Style/GuardClause:
   Enabled: false
 Style/FormatStringToken:
   Enabled: false
 Style/DoubleNegation:
   Enabled: false
+Style/IfUnlessModifier:
+  Enabled: false
+Style/EachWithObject:
+  Enabled: false
+Style/SafeNavigation:
+  Enabled: false
+Style/BlockDelimiters:
+  Enabled: false
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+Naming/VariableNumber:
+  Enabled: false
+Style/MultilineBlockChain:
+  Enabled: false
 
 # We aren't so brave to tackle all these issues right now
 Metrics/LineLength:
@@ -50,4 +66,6 @@ Style/MissingRespondToMissing:
   Enabled: false
 Naming/PredicateName:
   Enabled: false
+Security/YAMLLoad:
+  Enabled: false
 

gemfiles/rails_4_2.gemfile

@@ -4,8 +4,8 @@
 
 source 'https://rubygems.org'
 
+gem 'activemodel', '~> 4.2.0'
 gem 'nokogiri', '~> 1.6.8'
 gem 'pry-byebug', platforms: :ruby
-gem 'activemodel', '~> 4.2.0'
 
 gemspec path: '../'

gemfiles/rails_5_0.gemfile

@@ -4,7 +4,7 @@
 
 source 'https://rubygems.org'
 
-gem 'pry-byebug', platforms: :ruby
 gem 'activemodel', '~> 5.0.0'
+gem 'pry-byebug', platforms: :ruby
 
 gemspec path: '../'

gemfiles/rails_5_1.gemfile

@@ -4,7 +4,7 @@
 
 source 'https://rubygems.org'
 
-gem 'pry-byebug', platforms: :ruby
 gem 'activemodel', '~> 5.1.0'
+gem 'pry-byebug', platforms: :ruby
 
 gemspec path: '../'

gemfiles/rails_5_2.gemfile

@@ -4,7 +4,7 @@
 
 source 'https://rubygems.org'
 
-gem 'pry-byebug', platforms: :ruby
 gem 'activemodel', '~> 5.2.0'
+gem 'pry-byebug', platforms: :ruby
 
 gemspec path: '../'

lib/dynamoid/adapter.rb

@@ -155,6 +155,7 @@ def delete_table(table_name, options = {})
     # @since 0.2.0
     def method_missing(method, *args, &block)
       return benchmark(method, *args) { adapter.send(method, *args, &block) } if adapter.respond_to?(method)
+
       super
     end