Merge pull request #61 from dblock/rubocop-faraday-etc

Added Rubocop, upgraded Faraday, removed Gemfile.lock.

Author: dblock

.gitignore

@@ -15,3 +15,4 @@ test/tmp
 test/version_tmp
 tmp
 examples/william.rb
+Gemfile.lock

.rubocop.yml

@@ -0,0 +1,2 @@
+inherit_from: .rubocop_todo.yml
+

.rubocop_todo.yml

@@ -0,0 +1,47 @@
+# This configuration was generated by `rubocop --auto-gen-config`
+# on 2014-09-17 21:23:50 -0400 using RuboCop version 0.26.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 106
+
+# Offense count: 36
+# Configuration parameters: AllowURI.
+Metrics/LineLength:
+  Max: 140
+
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/MethodLength:
+  Max: 14
+
+# Offense count: 1
+Style/AsciiComments:
+  Enabled: false
+
+# Offense count: 3
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+# Offense count: 15
+Style/Documentation:
+  Enabled: false
+
+# Offense count: 1
+Style/DoubleNegation:
+  Enabled: false
+
+# Offense count: 3
+Style/Lambda:
+  Enabled: false
+
+# Offense count: 6
+# Configuration parameters: MaxSlashes.
+Style/RegexpLiteral:
+  Enabled: false

.travis.yml

@@ -1,3 +1,5 @@
 rvm:
   - 1.9.3
   - 2.0.0
+  - 2.1.2
+

Gemfile

@@ -11,5 +11,6 @@ gem 'pry'
 
 gem 'redcarpet'
 gem 'yard', '~> 0.8'
-gem 'yard-tomdoc', git: 'git://github.com/rubyworks/yard-tomdoc'
+gem 'yard-tomdoc'
 gem 'simplecov', require: false
+gem 'rubocop', '~> 0.26.0', require: false

Gemfile.lock

@@ -1,13 +1,13 @@
 guard 'minitest' do
-  watch(%r|^test/(.*)_test\.rb|)
-  watch(%r|^lib/(.*)([^/]+)\.rb|)     { |m| "test/#{m[1]}#{m[2]}_test.rb" }
-  watch(%r|^(.*)([^/]+)\.rb|)         { |m| "test/#{m[1]}#{m[2]}_test.rb" }
-  watch(%r|^test/test_helper\.rb|)    { "test" }
+  watch(%r{^test/(.*)_test\.rb})
+  watch(%r{^lib/(.*)([^/]+)\.rb})     { |m| "test/#{m[1]}#{m[2]}_test.rb" }
+  watch(%r{^(.*)([^/]+)\.rb})         { |m| "test/#{m[1]}#{m[2]}_test.rb" }
+  watch(%r{^test/test_helper\.rb})    { 'test' }
 end
 
 guard 'spinach' do
-  watch(%r|^features/(.*)\.feature|)
-  watch(%r|^features/steps/(.*)([^/]+)\.rb|) do |m|
+  watch(%r{^features/(.*)\.feature})
+  watch(%r{^features/steps/(.*)([^/]+)\.rb}) do |m|
     "features/#{m[1]}#{m[2]}.feature"
   end
 end

Guardfile

@@ -14,13 +14,12 @@ if ENV['COVERAGE']
 end
 
 require 'yard'
-YARD::Config.load_plugin('yard-tomdoc') 
+YARD::Config.load_plugin('yard-tomdoc')
 YARD::Rake::YardocTask.new do |t|
   t.files   = ['lib/**/*.rb']
   t.options = %w(-r README.md)
 end
 
-
 Bundler::GemHelper.install_tasks
 
 require 'rake/testtask'
@@ -37,4 +36,7 @@ task :spinach do
   ruby '-S spinach'
 end
 
-task default: [:test, :spinach]
+require 'rubocop/rake_task'
+RuboCop::RakeTask.new(:rubocop)
+
+task default: [:rubocop, :test, :spinach]

Rakefile

@@ -16,7 +16,7 @@ def games
   end
 
   def add_game(name)
-    client.links.submissions.post({name: name})
+    client.links.submissions.post(name: name)
   end
 
   def motd
@@ -32,9 +32,10 @@ def method_missing(method, *args, &block)
   end
 
   private
+
   def client
-    @client ||= Hyperclient::EntryPoint.new 'http://cs-api.heroku.com/api', 
-                {debug: false, headers: {'content-type' => 'application/json'}}
+    @client ||= Hyperclient::EntryPoint.new 'http://cs-api.heroku.com/api',
+                                            debug: false, headers: { 'content-type' => 'application/json' }
   end
 end
 
@@ -43,18 +44,17 @@ def print_links(links)
     if link.is_a?(Array)
       print_links(link)
     else
-      puts %{Found "#{name}" at "#{link.url}" }
+      puts %(Found "#{name}" at "#{link.url}" )
     end
   end
 end
 
 def print_games(games)
   games.each do |game|
-    puts %{Found "#{game.attributes['name']}" }
+    puts %(Found "#{game.attributes['name']}" )
   end
 end
 
-
 api = Cyberscore.new
 
 puts "Let's inspect the API:"
@@ -72,5 +72,5 @@ def print_games(games)
 print_links(api.links.news.links)
 puts "\n"
 
-puts "I like games!"
+puts 'I like games!'
 print_games(api.games)

examples/cyberscore.rb

@@ -4,17 +4,17 @@
 def print_resources(resources)
   resources.each do |name, resource|
     begin
-      puts %{Found #{name} at #{resource.url}}
+      puts %(Found #{name} at #{resource.url})
     rescue
-      puts %{Found #{name}}
+      puts %(Found #{name})
     end
   end
 end
 
 def print_attributes(attributes)
-  puts "-----------------------------"
+  puts '-----------------------------'
   attributes.each do |attribute, value|
-    puts %{#{attribute}: #{value}}
+    puts %(#{attribute}: #{value})
   end
 end
 
@@ -31,9 +31,9 @@ def print_attributes(attributes)
 puts "Let's see what stats we have:"
 print_attributes(api.embedded.stats.attributes)
 
-products = api.links["http://hal-shop.heroku.com/rels/products"].resource
+products = api.links['http://hal-shop.heroku.com/rels/products'].resource
 
-puts 
+puts
 puts "And what's the inventory of products?"
 puts products.attributes['inventory_size']