skeptic 0.0.2 → 0.0.3

data/VERSION

@@ -1 +1 @@
-0.0.2
+0.0.3

data/lib/skeptic/rules/lines_per_method.rb

@@ -33,9 +33,17 @@ module Skeptic
       private
 
       on :class do |name, parent, body|
-        class_name = [env[:class], extract_name(name)].compact.join('::')
+        class_name = [env[:module], extract_name(name)].compact.join('::')
 
-        env.scoped :class => class_name do
+        env.scoped :module => class_name do
+          visit body
+        end
+      end
+
+      on :module do |name, body|
+        module_name = [env[:module], extract_name(name)].compact.join('::')
+
+        env.scoped :module => module_name do
           visit body
         end
       end
@@ -48,7 +56,22 @@ module Skeptic
 
           lines = env[:line_numbers].uniq.compact.length
 
-          full_name = "#{env[:class]}##{env[:method]}"
+          full_name = "#{env[:module]}##{env[:method]}"
+          @line_counts[full_name] = lines + @line_counts.fetch(full_name, 0)
+        end
+      end
+
+      on :defs do |target, separator, name, params, body|
+        method_name = extract_name(name)
+        class_name  = extract_name(target)
+        class_name  = env[:module] if class_name == 'self'
+
+        env.scoped method: method_name, line_numbers: [] do
+          visit body
+
+          lines = env[:line_numbers].uniq.compact.length
+
+          full_name = "#{class_name}.#{method_name}"
           @line_counts[full_name] = lines + @line_counts.fetch(full_name, 0)
         end
       end

data/lib/skeptic/sexp_visitor.rb

@@ -25,7 +25,7 @@ module Skeptic
           type, *args = *sexp
           handler = self.class.handlers[type]
 
-          with_sexp_type(type) { instance_exec *args, &handler }
+          with_sexp_type(type) { instance_exec(*args, &handler) }
         else
           range = sexp[0].kind_of?(Symbol) ? 1..-1 : 0..-1
 
@@ -57,6 +57,8 @@ module Skeptic
           when :var_ref then extract_name(first)
           when :@const then first
           when :@ident then first
+          when :@kw then first
+          when :@op then first
           else '<unknown>'
         end
       end

data/skeptic.gemspec

@@ -0,0 +1,83 @@
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = "skeptic"
+  s.version = "0.0.3"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["Stefan Kanev"]
+  s.date = "2012-10-25"
+  s.description = "An experimental, half-assed, bug-ridden and highly opinionated code analyzer."
+  s.email = "stefan.kanev@gmail.com"
+  s.executables = ["skeptic"]
+  s.extra_rdoc_files = [
+    "LICENSE.txt",
+    "README.rdoc"
+  ]
+  s.files = [
+    ".document",
+    ".rspec",
+    ".rvmrc",
+    "Gemfile",
+    "LICENSE.txt",
+    "README.rdoc",
+    "Rakefile",
+    "VERSION",
+    "bin/skeptic",
+    "features/skeptic.feature",
+    "lib/skeptic.rb",
+    "lib/skeptic/critic.rb",
+    "lib/skeptic/environment.rb",
+    "lib/skeptic/rule_table.rb",
+    "lib/skeptic/rules.rb",
+    "lib/skeptic/rules/check_syntax.rb",
+    "lib/skeptic/rules/line_length.rb",
+    "lib/skeptic/rules/lines_per_method.rb",
+    "lib/skeptic/rules/max_nesting_depth.rb",
+    "lib/skeptic/rules/methods_per_class.rb",
+    "lib/skeptic/rules/no_semicolons.rb",
+    "lib/skeptic/rules/no_trailing_whitespace.rb",
+    "lib/skeptic/scope.rb",
+    "lib/skeptic/sexp_visitor.rb",
+    "skeptic.gemspec"
+  ]
+  s.homepage = "http://github.com/skanev/skeptic"
+  s.licenses = ["MIT"]
+  s.require_paths = ["lib"]
+  s.rubygems_version = "1.8.24"
+  s.summary = "Skeptic points out annoying things in your code"
+
+  if s.respond_to? :specification_version then
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+      s.add_runtime_dependency(%q<trollop>, [">= 1.16.2"])
+      s.add_development_dependency(%q<rspec>, [">= 0"])
+      s.add_development_dependency(%q<cucumber>, [">= 0"])
+      s.add_development_dependency(%q<aruba>, [">= 0"])
+      s.add_development_dependency(%q<bundler>, [">= 0"])
+      s.add_development_dependency(%q<jeweler>, [">= 0"])
+      s.add_development_dependency(%q<simplecov>, [">= 0"])
+    else
+      s.add_dependency(%q<trollop>, [">= 1.16.2"])
+      s.add_dependency(%q<rspec>, [">= 0"])
+      s.add_dependency(%q<cucumber>, [">= 0"])
+      s.add_dependency(%q<aruba>, [">= 0"])
+      s.add_dependency(%q<bundler>, [">= 0"])
+      s.add_dependency(%q<jeweler>, [">= 0"])
+      s.add_dependency(%q<simplecov>, [">= 0"])
+    end
+  else
+    s.add_dependency(%q<trollop>, [">= 1.16.2"])
+    s.add_dependency(%q<rspec>, [">= 0"])
+    s.add_dependency(%q<cucumber>, [">= 0"])
+    s.add_dependency(%q<aruba>, [">= 0"])
+    s.add_dependency(%q<bundler>, [">= 0"])
+    s.add_dependency(%q<jeweler>, [">= 0"])
+    s.add_dependency(%q<simplecov>, [">= 0"])
+  end
+end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: skeptic
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-23 00:00:00.000000000 Z
+date: 2012-10-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: trollop
@@ -156,6 +156,7 @@ files:
 - lib/skeptic/rules/no_trailing_whitespace.rb
 - lib/skeptic/scope.rb
 - lib/skeptic/sexp_visitor.rb
+- skeptic.gemspec
 homepage: http://github.com/skanev/skeptic
 licenses:
 - MIT
@@ -171,7 +172,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1704281601069197093
+      hash: -47347213712121151
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: