Merge pull request #3 from boost/skip-cli-wrapper-dependency

Skip cli wrapper dependency

Author: isabel-anastasiadis-boost

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    codeclimate_diff (0.1.4)
+    codeclimate_diff (0.1.5)
       colorize
       json
       optparse

README.md

@@ -9,11 +9,7 @@ NOTE: similar code will only work correctly if you run a diff on all the files i
 
 ## Installation
 
-1. Install the codeclimate cli:
-      ```bash
-      brew tap codeclimate/formulae
-      brew install codeclimate
-      ```
+1. Make sure docker is installed and running
 
 2. Add a `.codeclimate.yml` config file eg:
       ```yml
@@ -90,7 +86,7 @@ NOTE: similar code will only work correctly if you run a diff on all the files i
 
 4. Add a `.codecimate_diff.yml` configuration file
       ```
-      main_branch_name: main # defaults to main
+      main_branch_name: master # defaults to main
       ```
 
 5. Install the gem

lib/codeclimate_diff/codeclimate_wrapper.rb

@@ -6,7 +6,18 @@
 module CodeclimateDiff
   class CodeclimateWrapper
     def run_codeclimate(filename = "")
-      `codeclimate analyze -f json #{filename}`
+      `docker run \
+        --interactive --tty --rm \
+        --env CODECLIMATE_CODE="$PWD" \
+        --volume "$PWD":/code \
+        --volume /var/run/docker.sock:/var/run/docker.sock \
+        --volume /tmp/cc:/tmp/cc \
+        codeclimate/codeclimate analyze -f json #{filename}`
+    end
+
+    def pull_latest_image
+      puts "Downloading latest codeclimate docker image..."
+      `docker pull codeclimate/codeclimate`
     end
   end
 end

lib/codeclimate_diff/runner.rb

@@ -13,8 +13,6 @@ def self.calculate_changed_filenames(pattern)
       extra_grep_filter = pattern ? " | grep '#{pattern}'" : ""
       branch_name = CodeclimateDiff.configuration["main_branch_name"] || "main"
       files_changed_str = `git diff --name-only #{branch_name} | grep --invert-match spec/ | grep --extended-regexp '.js$|.rb$'#{extra_grep_filter}`
-      puts "Files changed on branch: #{files_changed_str}"
-
       files_changed_str.split("\n")
     end
 
@@ -46,13 +44,17 @@ def self.calculate_preexisting_issues_in_changed_files(changed_filenames)
     end
 
     def self.generate_baseline
+      CodeclimateWrapper.new.pull_latest_image
+
       puts "Generating the baseline.  Should take about 5 minutes..."
       result = CodeclimateWrapper.new.run_codeclimate
       File.write("codeclimate_diff_baseline.json", result)
       puts "Done!"
     end
 
     def self.run_diff_on_branch(pattern, show_preexisting: true)
+      CodeclimateWrapper.new.pull_latest_image
+
       changed_filenames = calculate_changed_filenames(pattern)
 
       changed_file_issues = calculate_issues_in_changed_files(changed_filenames)

lib/codeclimate_diff/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CodeclimateDiff
-  VERSION = "0.1.4"
+  VERSION = "0.1.5"
 end