ice_nine 0.8.0 → 0.9.0

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    MTFlNzA3NDRhNmQwNjNlZjc4ZjA0ZWYxMjE0ZWNlMDY3NDQ1MjNlOA==
+  data.tar.gz: !binary |-
+    YTZiZGQzNThmNmQ1ZjZhNTRlYjBmOTI5MWM1MGVhNjE2YzhhMjIwNQ==
+!binary "U0hBNTEy":
+  metadata.gz: !binary |-
+    NjQ5YWYxMDEwYjhjMjFhNDA5MzdiMTlhNzAwNmJlMDVkNWQwNDk5MmFmYTdk
+    ZmNhZjM3YjQ1NjE5MjBmNjIzMWFlZmVlYjQyYTIxMmMzZDRjMDRhMGRjODVk
+    MWE3OTRiMjY3ZDhmMzdkYzk5Yjc3Y2Y5ODg0NWZiM2U0NWZlZTY=
+  data.tar.gz: !binary |-
+    OGNjMWJmMjc2YTNkMWQxNWI2NGI0MTUxOTE0ZWYzYzY3YjdkMTQ4NGRjYzNm
+    NzgzMWNlMDAyMGI3NGUyNzRlZjkyZjkzNWVkYTMwMWJmMWFhMmU0MjUzM2Q0
+    OTM2YzdjYmJkYTlkYzA1Yjk1MTljMTg5OThlZWZhMzJjMzdkNjM=

data/.rspec

@@ -0,0 +1,5 @@
+--color
+--format progress
+--profile
+--warnings
+--order random

data/.travis.yml

@@ -1,20 +1,31 @@
 language: ruby
 before_install: gem install bundler
-bundler_args: --without yard guard metrics benchmarks
-script: "bundle exec rake spec"
+bundler_args: --without yard guard benchmarks
+script: "bundle exec rake ci"
 rvm:
-  - ree
-  - 1.8.7
-  - 1.9.2
   - 1.9.3
   - 2.0.0
   - ruby-head
-  - jruby-18mode
-  - jruby-19mode
-  - jruby-head
-  - rbx-18mode
-  - rbx-19mode
+#  - rbx-19mode
 notifications:
-  irc: "irc.freenode.org#datamapper"
+  irc:
+    channels:
+      - irc.freenode.org#rom-rb
+    on_success: never
+    on_failure: change
   email:
-    - dan.kubb@gmail.com
+    recipients:
+      - dan.kubb@gmail.com
+    on_success: never
+    on_failure: change
+matrix:
+  include:
+    - rvm: jruby-19mode
+      env: JRUBY_OPTS="$JRUBY_OPTS --debug"
+    - rvm: jruby-head
+      env: JRUBY_OPTS="$JRUBY_OPTS --debug"
+  allow_failures:
+    # broken on travis
+    - rvm: ruby-head
+    # simplecov broken
+    - rvm: rbx-19mode

data/Gemfile

@@ -4,70 +4,8 @@ source 'https://rubygems.org'
 
 gemspec
 
-group :development do
-  gem 'rake',  '~> 10.0.4'
-  gem 'rspec', '~> 1.3.2'
-  gem 'yard',  '~> 0.8.6.1'
+group :development, :test do
+  gem 'devtools', git: 'https://github.com/rom-rb/devtools.git'
 end
 
-group :yard do
-  gem 'kramdown', '~> 1.0.1'
-end
-
-group :guard do
-  gem 'guard',         '~> 1.8.0'
-  gem 'guard-bundler', '~> 1.0.0'
-  gem 'guard-rspec',   '~> 1.2.1'
-
-  # file system change event handling
-  gem 'listen',     '~> 1.2.1'
-  gem 'rb-fchange', '~> 0.0.6', :require => false
-  gem 'rb-fsevent', '~> 0.9.3', :require => false
-  gem 'rb-inotify', '~> 0.9.0', :require => false
-
-  # notification handling
-  gem 'libnotify',               '~> 0.8.0', :require => false
-  gem 'rb-notifu',               '~> 0.0.4', :require => false
-  gem 'terminal-notifier-guard', '~> 1.5.3', :require => false
-end
-
-group :metrics do
-  gem 'backports', '~> 3.3', '>= 3.3.2'
-  gem 'coveralls', '~> 0.6.7'
-  gem 'flay',      '~> 1.4.3'
-  gem 'flog',      '~> 2.5.3'
-  gem 'simplecov', '~> 0.7.1'
-  gem 'roodi',     '~> 2.2.0'
-  gem 'yardstick', '~> 0.9.2'
-
-  platforms :ruby_19 do
-    gem 'yard-spellcheck', '~> 0.1.5'
-  end
-
-  platforms :mri_18 do
-    gem 'arrayfields',          '~> 4.9.0'  # for metric_fu
-    gem 'fattr',                '~> 2.2.0'  # for metric_fu
-    gem 'heckle',               '~> 1.4.3'
-    gem 'json',                 '~> 1.8.0'  # for metric_fu rake task
-    gem 'map',                  '~> 6.5.1'  # for metric_fu
-    gem 'metric_fu',            '~> 2.1.1'
-    gem 'mspec',                '~> 1.5.17'
-    gem 'rails_best_practices', '= 1.13.3'  # for metric_fu
-    gem 'rcov',                 '~> 1.0.0'
-    gem 'ruby2ruby',            '= 1.2.2'   # for heckle
-  end
-
-  platforms :rbx do
-    gem 'pelusa', '~> 0.2.2'
-  end
-end
-
-group :benchmarks do
-  gem 'rbench', '~> 0.2.3'
-end
-
-platform :jruby do
-  group :jruby do
-    gem 'jruby-openssl', '~> 0.8.5'
-  end
-end
+eval_gemfile 'Gemfile.devtools'

data/Gemfile.devtools

@@ -0,0 +1,55 @@
+# encoding: utf-8
+
+group :development do
+  gem 'rake',  '~> 10.1.0'
+  gem 'rspec', '~> 2.14.1'
+  gem 'yard',  '~> 0.8.7'
+end
+
+group :yard do
+  gem 'kramdown', '~> 1.1.0'
+end
+
+group :guard do
+  gem 'guard',         '~> 1.8.1'
+  gem 'guard-bundler', '~> 1.0.0'
+  gem 'guard-rspec',   '~> 3.0.2'
+  gem 'guard-rubocop', '~> 0.2.0'
+#  gem 'guard-mutant',  '~> 0.0.1'
+
+  # file system change event handling
+  gem 'listen',     '~> 1.3.0'
+  gem 'rb-fchange', '~> 0.0.6', require: false
+  gem 'rb-fsevent', '~> 0.9.3', require: false
+  gem 'rb-inotify', '~> 0.9.0', require: false
+
+  # notification handling
+  gem 'libnotify',               '~> 0.8.0', require: false
+  gem 'rb-notifu',               '~> 0.0.4', require: false
+  gem 'terminal-notifier-guard', '~> 1.5.3', require: false
+end
+
+group :metrics do
+  gem 'coveralls', '~> 0.6.7'
+  gem 'flay',      '~> 2.4.0'
+  gem 'flog',      '~> 4.1.1'
+  gem 'reek',      '~> 1.3.2'
+  gem 'rubocop',   '~> 0.11.0'
+  gem 'simplecov', '~> 0.7.1'
+  gem 'yardstick', '~> 0.9.7', git: 'https://github.com/dkubb/yardstick.git'
+
+  platforms :ruby_19, :ruby_20 do
+#    gem 'mutant', git: 'https://github.com/mbj/mutant.git'
+    gem 'yard-spellcheck', '~> 0.1.5'
+  end
+end
+
+group :benchmarks do
+  gem 'rbench', '~> 0.2.3'
+end
+
+platform :jruby do
+  group :jruby do
+    gem 'jruby-openssl', '~> 0.8.5'
+  end
+end

data/Rakefile

@@ -1,8 +1,4 @@
 # encoding: utf-8
 
-require 'rake'
-
-FileList['tasks/**/*.rake'].each { |task| import task }
-
-desc 'Default: run all specs'
-task :default => :spec
+require 'devtools'
+Devtools.init_rake_tasks

data/config/devtools.yml

@@ -0,0 +1,2 @@
+---
+unit_test_timeout: 0.1

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
-threshold: 6
-total_score: 37.0
+threshold: 4
+total_score: 22.0

data/config/flog.yml

@@ -1,2 +1,2 @@
 ---
-threshold: 9.7
+threshold: 8.0

data/config/mutant.yml

@@ -0,0 +1,3 @@
+---
+name: ice_nine
+namespace: IceNine

data/config/{site.reek → reek.yml}

@@ -1,91 +1,106 @@
 ---
-UncommunicativeParameterName:
-  accept: []
-  exclude: []
+Attribute:
   enabled: true
-  reject:
-  - !ruby/regexp /^.$/
-  - !ruby/regexp /[0-9]$/
-  - !ruby/regexp /[A-Z]/
-LargeClass:
-  max_methods: 1
   exclude: []
+BooleanParameter:
   enabled: true
-  max_instance_variables: 1
-UncommunicativeMethodName:
-  accept: []
   exclude: []
+ClassVariable:
   enabled: true
-  reject:
-  - !ruby/regexp /^[a-z]$/
-  - !ruby/regexp /[0-9]$/
-  - !ruby/regexp /[A-Z]/
-LongParameterList:
-  max_params: 1
   exclude: []
+ControlParameter:
   enabled: true
-  overrides: {}
-FeatureEnvy:
   exclude: []
+DataClump:
   enabled: true
-ClassVariable:
   exclude: []
+  max_copies: 2
+  min_clump_size: 2
+DuplicateMethodCall:
   enabled: true
-BooleanParameter:
   exclude: []
+  max_calls: 1
+  allow_calls: []
+FeatureEnvy:
   enabled: true
+  exclude: []
 IrresponsibleModule:
+  enabled: true
   exclude: []
+LongParameterList:
   enabled: true
-UncommunicativeModuleName:
-  accept: []
   exclude: []
+  max_params: 2
+  overrides:
+    initialize:
+      max_params: 3
+LongYieldList:
   enabled: true
-  reject:
-  - !ruby/regexp /^.$/
-  - !ruby/regexp /[0-9]$/
+  exclude: []
+  max_params: 2
 NestedIterators:
+  enabled: true
+  exclude: []
+  max_allowed_nesting: 1
   ignore_iterators: []
+NilCheck:
+  enabled: true
   exclude: []
+RepeatedConditional:
   enabled: true
-  max_allowed_nesting: 1
-LongMethod:
-  max_statements: 5
   exclude: []
+  max_ifs: 1
+TooManyInstanceVariables:
   enabled: true
-Duplication:
-  allow_calls: []
   exclude: []
+  max_instance_variables: 3
+TooManyMethods:
   enabled: true
-  max_calls: 1
-UtilityFunction:
-  max_helper_calls: 0
   exclude: []
+  max_methods: 10
+TooManyStatements:
+  enabled: true
+  exclude:
+  - IceNine::Freezer::Hash#self.deep_freeze
+  - IceNine::Freezer::Range#self.deep_freeze
+  - IceNine::RecursionGuard#self.guard
+  - each
+  max_statements: 2
+UncommunicativeMethodName:
   enabled: true
-Attribute:
   exclude: []
-  enabled: false
-UncommunicativeVariableName:
+  reject:
+  - !ruby/regexp /^[a-z]$/
+  - !ruby/regexp /[0-9]$/
+  - !ruby/regexp /[A-Z]/
   accept: []
+UncommunicativeModuleName:
+  enabled: true
   exclude: []
+  reject:
+  - !ruby/regexp /^.$/
+  - !ruby/regexp /[0-9]$/
+  accept: []
+UncommunicativeParameterName:
   enabled: true
+  exclude: []
   reject:
   - !ruby/regexp /^.$/
   - !ruby/regexp /[0-9]$/
   - !ruby/regexp /[A-Z]/
-SimulatedPolymorphism:
-  exclude: []
+  accept: []
+UncommunicativeVariableName:
   enabled: true
-  max_ifs: 1
-DataClump:
   exclude: []
+  reject:
+  - !ruby/regexp /^.$/
+  - !ruby/regexp /[0-9]$/
+  - !ruby/regexp /[A-Z]/
+  accept: []
+UnusedParameters:
   enabled: true
-  max_copies: 0
-  min_clump_size: 0
-ControlCouple:
   exclude: []
+UtilityFunction:
   enabled: true
-LongYieldList:
-  max_params: 0
   exclude: []
-  enabled: true
+  max_helper_calls: 0

data/config/rubocop.yml

@@ -0,0 +1,62 @@
+AllCops:
+  Includes:
+    - '**/*.rake'
+    - 'Gemfile'
+    - 'Gemfile.devtools'
+  Excludes:
+    - '**/vendor/**'
+    - '**/benchmarks/**'
+
+# Avoid parameter lists longer than five parameters.
+ParameterLists:
+  Max: 3
+  CountKeywordArgs: true
+
+# Avoid more than `Max` levels of nesting.
+BlockNesting:
+  Max: 3
+
+# Align with the style guide.
+CollectionMethods:
+  PreferredMethods:
+    collect:  'map'
+    inject:   'reduce'
+    find:     'detect'
+    find_all: 'select'
+
+# Do not force public/protected/private keyword to be indented at the same
+# level as the def keyword. My personal preference is to outdent these keywords
+# because I think when scanning code it makes it easier to identify the
+# sections of code and visually separate them. When the keyword is at the same
+# level I think it sort of blends in with the def keywords and makes it harder
+# to scan the code and see where the sections are.
+AccessControl:
+  Enabled: false
+
+# Limit line length
+LineLength:
+  Max: 79
+
+# Disable documentation checking until a class needs to be documented once
+Documentation:
+  Enabled: false
+
+# Do not always use &&/|| instead of and/or.
+AndOr:
+  Enabled: false
+
+# Do not favor modifier if/unless usage when you have a single-line body
+IfUnlessModifier:
+  Enabled: false
+
+# Allow case equality operator (in limited use within the specs)
+CaseEquality:
+  Enabled: false
+
+# Constants do not always have to use SCREAMING_SNAKE_CASE
+ConstantName:
+  Enabled: false
+
+# Not all trivial readers/writers can be defined with attr_* methods
+TrivialAccessors:
+  Enabled: false