codeclimate 0.11.0 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9516f4110843547b299db402d54230e61bb63b80
-  data.tar.gz: 3fc082327dd10c49850935b9a7f2cdf3d446ec0b
+  metadata.gz: a802a87b63c111d67efe9d1663f178e29248ec13
+  data.tar.gz: 6d583607b2efdcc87ec46206a25147de6e1c3d0e
 SHA512:
-  metadata.gz: e890e440bccebf48ea429e7cd542b1a2108edba0211b221cc10e91f91e03de92b5619097c8e2154c9a2009b71c24637d2c159dcdf4eb08670ed885fd9cc328a3
-  data.tar.gz: a9a276927f30546b55d24ed7c3e79737a444c946f22b2aae940975f6531e3536747a3b305a89d2ccdae177d23076d336b6e2ecb25319a89859d154386ca41cdb
+  metadata.gz: 9295ad43adf6fb0ee295b275e25e6ee0c7e75ae6ee0cc6f077fc9e9e76d1a79d5f408c22104255ef4de8d83d89dbae06141bf5e8a8b10c55cd133931f1629a9e
+  data.tar.gz: 4508c8bcff6f12399ba312dbdb11cf35e10f007b26947c58a870430061ed7f851d0803a1373fa7f059c05e7fcabc8e192dcff13af8ab500cf564cbc4a0b448a4

data/config/engines.yml

@@ -9,19 +9,44 @@
 # the `default_ratings_paths` globs, which are used during analysis to determine
 # which files should be rated.
 #
-rubocop:
-  image: codeclimate/codeclimate-rubocop
-  description: A Ruby static code analyzer, based on the community Ruby style guide.
-  community: false
-  upgrade_languages:
-   - Ruby
+brakeman:
+  image: codeclimate/codeclimate-brakeman
+  description: Static analysis tool which checks Ruby on Rails applications for security vulnerabilities.
+  community: true
   enable_regexps:
     - \.rb$
   default_ratings_paths:
+    - "app/**"
     - "**.rb"
+bundler-audit:
+  image: codeclimate/codeclimate-bundler-audit
+  description: Patch-level verification for Bundler.
+  community: false
+  upgrade_languages:
+    - Ruby
+  enable_regexps:
+    - ^Gemfile\.lock$
+  default_ratings_paths:
+    - Gemfile.lock
+csslint:
+  image: codeclimate/codeclimate-csslint
+  description: Automated linting of Cascading Stylesheets.
+  community: false
+  enable_regexps:
+    - \.css$
+  default_ratings_paths:
+    - "**.css"
+coffeelint:
+  image: codeclimate/codeclimate-coffeelint
+  description: A style checker for CoffeeScript.
+  community: false
+  enable_regexps:
+    - \.coffee$
+  default_ratings_paths:
+    - "**.coffee"
 duplication:
   image: codeclimate/codeclimate-duplication
-  description: Structural duplication detection for Ruby, Python, JavaScript, and PHP
+  description: Structural duplication detection for Ruby, Python, JavaScript, and PHP.
   community: false
   enable_regexps:
   default_ratings_paths:
@@ -31,9 +56,21 @@ duplication:
       - javascript
       - python
       - php
+eslint:
+  image: codeclimate/codeclimate-eslint
+  description: A JavaScript/JSX linting utility.
+  community: false
+  upgrade_languages:
+    - JavaScript
+  enable_regexps:
+    - \.js$
+    - \.jsx$
+  default_ratings_paths:
+    - "**.js"
+    - "**.jsx"
 gofmt:
   image: codeclimate/codeclimate-gofmt
-  description: gofmt
+  description: Checks the formatting of Go programs.
   community: true
   enable_regexps:
     - \.go$
@@ -41,7 +78,7 @@ gofmt:
     - "**.go"
 golint:
   image: codeclimate/codeclimate-golint
-  description: golint
+  description: A linter for Go.
   community: true
   enable_regexps:
     - \.go$
@@ -49,67 +86,36 @@ golint:
     - "**.go"
 govet:
   image: codeclimate/codeclimate-govet
-  description: govet
+  description: Reports suspicious constructs in Go programs.
   community: true
   enable_regexps:
     - \.go$
   default_ratings_paths:
     - "**.go"
-coffeelint:
-  image: codeclimate/codeclimate-coffeelint
-  description: A style checker for CoffeeScript
-  community: false
-  enable_regexps:
-    - \.coffee$
-  default_ratings_paths:
-    - "**.coffee"
-eslint:
-  image: codeclimate/codeclimate-eslint
-  description: A JavaScript/JSX linting utility
-  community: false
-  upgrade_languages:
-    - JavaScript
-  enable_regexps:
-    - \.js$
-    - \.jsx$
-  default_ratings_paths:
-    - "**.js"
-    - "**.jsx"
-csslint:
-  image: codeclimate/codeclimate-csslint
-  description: Automated linting of Cascading Stylesheets
+fixme:
+  image: codeclimate/codeclimate-fixme
+  description: Finds FIXME, TODO, HACK, etc. comments.
   community: false
   enable_regexps:
-    - \.css$
-  default_ratings_paths:
-    - "**.css"
-watson:
-  image: codeclimate/codeclimate-watson
-  description: A young Ember Doctor to help you fix your code.
+    - .+
+  default_ratings_paths: []
+foodcritic:
+  image: codeclimate/codeclimate-foodcritic
+  description: Lint tool for Chef cookbooks.
   community: true
   enable_regexps:
   default_ratings_paths:
-    - "app/**"
-rubymotion:
-  image: codeclimate/codeclimate-rubymotion
-  description: Rubymotion-specific rubocop checks
+pep8:
+  image: codeclimate/codeclimate-pep8
+  description: Static analysis tool to check Python code against the style conventions outlined in PEP-8.
   community: true
   enable_regexps:
+    - \.py$
   default_ratings_paths:
-    - "**.rb"
-bundler-audit:
-  image: codeclimate/codeclimate-bundler-audit
-  description: Patch-level verification for Bundler
-  community: false
-  upgrade_languages:
-    - Ruby
-  enable_regexps:
-    - ^Gemfile\.lock$
-  default_ratings_paths:
-    - Gemfile.lock
+    - "**.py"
 phpcodesniffer:
   image: codeclimate/codeclimate-phpcodesniffer
-  description: PHP Code Sniffer
+  description: Detects violations of a defined set of coding standards in PHP.
   community: false
   upgrade_languages:
     - PHP
@@ -123,7 +129,7 @@ phpcodesniffer:
     - "**.inc"
 phpmd:
   image: codeclimate/codeclimate-phpmd
-  description: PHP Mess Detector
+  description: A PHP static analysis tool.
   community: false
   upgrade_languages:
     - PHP
@@ -135,29 +141,48 @@ phpmd:
     - "**.php"
     - "**.module"
     - "**.inc"
-fixme:
-  image: codeclimate/codeclimate-fixme
-  description: Find FIXME, TODO, HACK, etc. comments
+radon:
+  image: codeclimate/codeclimate-radon
+  description: Python tool used to compute Cyclomatic Complexity.
+  community: true
+  enable_regexps:
+    - \.py$
+  default_ratings_paths:
+    - "**.py"
+requiresafe:
+  image: codeclimate/codeclimate-requiresafe
+  description: Security tool for Node.js dependencies.
+  community: true
+  enable_regexps:
+  default_ratings_paths:
+rubocop:
+  image: codeclimate/codeclimate-rubocop
+  description: A Ruby static code analyzer, based on the community Ruby style guide.
   community: false
+  upgrade_languages:
+   - Ruby
   enable_regexps:
-    - .+
-  default_ratings_paths: []
-foodcritic:
-  image: codeclimate/codeclimate-foodcritic
-  description: Lint tool for Chef cookbooks
+    - \.rb$
+  default_ratings_paths:
+    - "**.rb"
+rubymotion:
+  image: codeclimate/codeclimate-rubymotion
+  description: Rubymotion-specific rubocop checks.
   community: true
   enable_regexps:
   default_ratings_paths:
+    - "**.rb"
 scss-lint:
   image: codeclimate/codeclimate-scss-lint
-  description: Configurable tool for writing clean and consistent SCSS
+  description: Configurable tool for writing clean and consistent SCSS.
   community: true
   enable_regexps:
   default_ratings_paths:
     - "**.scss"
-requiresafe:
-  image: codeclimate/codeclimate-requiresafe
-  description: Security tool for Node.js dependencies
+watson:
+  image: codeclimate/codeclimate-watson
+  description: A young Ember Doctor to help you fix your code.
   community: true
   enable_regexps:
   default_ratings_paths:
+    - "app/**"

data/lib/cc/analyzer/engines_builder.rb

@@ -44,7 +44,7 @@ module CC
 
       def names_and_raw_engine_configs
         {}.tap do |ret|
-          @config.engines.each do |name, raw_engine_config|
+          (@config.engines || {}).each do |name, raw_engine_config|
             if raw_engine_config.enabled? && @registry.key?(name)
               ret[name] = raw_engine_config
             end

data/lib/cc/cli/config_generator.rb

@@ -17,6 +17,10 @@ module CC
         @engine_registry = engine_registry
       end
 
+      def can_generate?
+        true
+      end
+
       def eligible_engines
         return @eligible_engines if @eligible_engines
 
@@ -28,6 +32,10 @@ module CC
         end
       end
 
+      def errors
+        []
+      end
+
       def exclude_paths
         AUTO_EXCLUDE_PATHS.select { |path| filesystem.exist?(path) }
       end

data/lib/cc/cli/init.rb

@@ -9,13 +9,11 @@ module CC
 
       def run
         if !upgrade? && filesystem.exist?(CODECLIMATE_YAML)
-          say "Config file .codeclimate.yml already present.\nTry running 'validate-config' to check configuration."
+          fatal "Config file .codeclimate.yml already present.\nTry running 'validate-config' to check configuration."
         elsif upgrade? && engines_enabled?
-          say "--upgrade should not be used on a .codeclimate.yml configured for the Platform.\nTry running 'validate-config' to check configuration."
+          fatal "--upgrade should not be used on a .codeclimate.yml configured for the Platform.\nTry running 'validate-config' to check configuration."
         else
-          create_codeclimate_yaml
-          say "Config file .codeclimate.yml successfully #{config_generator.post_generation_verb}.\nEdit and then try running 'validate-config' to check configuration."
-          create_default_configs
+          generate_config
         end
       end
 
@@ -25,6 +23,19 @@ module CC
         @args.include?("--upgrade")
       end
 
+      def generate_config
+        unless config_generator.can_generate?
+          config_generator.errors.each do |error|
+            $stderr.puts colorize("ERROR: #{error}", :red)
+          end
+          fatal "Cannot generate .codeclimate.yml: please address above errors."
+        end
+
+        create_codeclimate_yaml
+        say "Config file .codeclimate.yml successfully #{config_generator.post_generation_verb}.\nEdit and then try running 'validate-config' to check configuration."
+        create_default_configs
+      end
+
       def create_codeclimate_yaml
         config = CC::CLI::Config.new
 

data/lib/cc/cli/upgrade_config_generator.rb

@@ -3,8 +3,16 @@ require "cc/cli/config_generator"
 module CC
   module CLI
     class UpgradeConfigGenerator < ConfigGenerator
+      def can_generate?
+        errors.blank?
+      end
+
+      def errors
+        existing_yaml.errors
+      end
+
       def exclude_paths
-        existing_yaml["exclude_paths"] || []
+        (existing_yaml.exclude_paths || []).map(&:to_s)
       end
 
       def post_generation_verb
@@ -23,11 +31,11 @@ module CC
       end
 
       def classic_languages
-        @classic_languages ||= existing_yaml["languages"].reject { |_, v| !v }.map(&:first)
+        @classic_languages ||= existing_yaml.languages.reject { |_, v| !v }.map(&:first)
       end
 
       def existing_yaml
-        @existing_yml ||= YAML.safe_load(File.read(CODECLIMATE_YAML))
+        @existing_yaml ||= CC::Yaml.parse(File.read(CODECLIMATE_YAML))
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.13.0
 platform: ruby
 authors:
 - Code Climate
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-28 00:00:00.000000000 Z
+date: 2015-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -50,14 +50,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: 0.4.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: 0.4.0
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement