face_control 0.8.1 → 0.8.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3b82b2b2726396b35762b53671f25666dc1562c7
-  data.tar.gz: b2803733c726bb02fd02931dc58cff928d59ec98
+SHA256:
+  metadata.gz: 325aba4c4fdce156c0dae24d9c5cf10b2993b4b70e0fcf184d3645cea22f5563
+  data.tar.gz: 700af54ea35e67f76d9245acb0fc5e63dfaa766295b357d27d71cf5e0a7de3a2
 SHA512:
-  metadata.gz: 4b4b10c4feceace242f85e576e020f102783f1bbf2527c14c1897ba91b93d9069a30b4adf91a1aa20956be48e9bcea2d4b428cebd77b7b7c9331fe64b90b5720
-  data.tar.gz: 58f631ad6606b34b18f204780024d9233c2fb1a715876e4dc33123ea12b2337a7c22b1a4a6235aa1f0b0abd45a617678d1a469888cf3df012ef382dcd3f5b8ef
+  metadata.gz: d5df85b4788e83c52f3cb051fee19d5137794b3ae9096e2bcc626b0585783515bb1479d52bf9c1cc2fab1d0cd829cbbc69120ccdf48f95e9e686144e4a047163
+  data.tar.gz: 308cb4a0b3d91dc19b1da72b537ed7dbd17f327554bbbc091e7ef16cd895f4828391490843ae3e63c554e0c73d4ce7935bb86131120d176b094f2390a70c9f0f

data/.gitignore

@@ -8,3 +8,4 @@ doc/
 pkg/
 spec/reports/
 tmp/
+.idea

data/.vexor.yml

@@ -1,5 +1,4 @@
 rvm:
-- 1.9
 - 2.0
 - 2.1
 - 2.2

data/README.md

@@ -1,76 +1,94 @@
+# Face Control
+
 [![Gem Version](https://img.shields.io/gem/v/face_control.svg)](https://rubygems.org/gems/face_control)
 [![Code Climate](https://img.shields.io/codeclimate/github/vassilevsky/face_control.svg)](https://codeclimate.com/github/vassilevsky/face_control/code)
 [![Vexor](https://ci.vexor.io/projects/126da196-c8e6-46f0-8bc7-b5f8f4b49732/status.svg)](https://ci.vexor.io/ui/projects/126da196-c8e6-46f0-8bc7-b5f8f4b49732/builds)
 [![Coveralls](https://img.shields.io/coveralls/vassilevsky/face_control.svg)](https://coveralls.io/github/vassilevsky/face_control)
 [![VersionEye](https://img.shields.io/versioneye/d/ruby/face_control.svg)](https://www.versioneye.com/ruby/face_control)
 
-# Face Control
+Run static analysis of pull requests in [Bitbucket Server][] (formerly Stash)
+and comment on problems in added lines.
 
-Comment on added lines of pull requests in [Atlassian Stash][].
-Take comments from static checkers reports.
-(Currently supports [RuboCop][] and [CoffeeLint][].)
+Currently supports [RuboCop][] and [CoffeeLint][] and also checks for
+TODOs and FIXMEs.
 
 Inspired by [Hound][].
 
 ## Installation
 
-    gem install face_control
+```bash
+gem install face_control
+```
+
+You also need to have CoffeeLint installed and available in `PATH`.
 
 ## Usage
 
-    face-control <project> <repository> <pull_request_id>
+```bash
+face-control <project> <repository> <pull_request_id>
+```
+
+It's natural to run this on a continuous integration server (see “[Example](#example)” below).
+
+If you don't want to receive RuboCop comments with certain severity level,
+pass the severity in the `--skip-severity` option like so:
+
+```bash
+face-control --skip-severity convention <project> <repository> <pull_request_id>
+```
 
-It's natural to run this on a continuous integration server.
-For example, here's a [Jenkins][] project setup:
+Instead of `--skip-severity` you can use just `-S`.
 
-* Source Code Management
-  * Git
-    * Repositories
-      * Refspec:
+You can also pass multiple severity levels as a comma-separated list:
 
-              +refs/pull-requests/*:refs/remotes/origin/pull-requests/*
+```bash
+face-control -S convention,refactor <project> <repository> <pull_request_id>
+```
 
-        (make Jenkins fetch otherwise ignored Stash-created branches)
+`face-control` uses the same configuration file (`~/.stashconfig.yml`)
+as the official [Bitbucket Server Command Line Tools][]
+to connect to your Stash instance.
 
-    * Branches to build
-      * Branch Specifier:
+## Example
 
-              origin/pull-requests/*/merge
+Here's a [Jenkins][] project setup as an example:
 
-        (merge results of open non-conflicting pull requests)
+_Source Code Management → Git → Repositories → Refspec:_
 
-* Build
-  * Execute shell
-    * Command
+```
++refs/pull-requests/*:refs/remotes/origin/pull-requests/*
+```
 
-            export PULL_REQUEST_ID=`echo $GIT_BRANCH | cut -d / -f 3`
+It makes Jenkins fetch otherwise ignored Stash-created branches.
 
-            gem install rubocop face_control
-            npm install -g coffeelint
+_Source Code Management → Git → Branches to build → Branch Specifier:_
 
-            face-control <project> <repository> $PULL_REQUEST_ID
+```
+origin/pull-requests/*/merge
+```
 
-If you don't want to receive RuboCop comments with certain severity level,
-pass the severity in the `--skip-severity` option like so:
+Merge results of open non-conflicting pull requests.
 
-    face-control --skip-severity convention <project> <repository> <pull_request_id>
+_Build → Execute shell → Command:_
 
-You can use just `-S`.
-You can also pass multiple severity levels as a comma-separated list:
+```bash
+export PULL_REQUEST_ID=`echo $GIT_BRANCH | cut -d / -f 3`
 
-    face-control -S convention,refactor <project> <repository> <pull_request_id>
+gem install rubocop face_control
+npm install -g coffeelint
 
-`face-control` uses the same configuration file (`~/.stashconfig.yml`)
-as the official [Atlassian Stash Command Line Tools][]
-to connect to your Stash instance.
+face-control <project> <repository> $PULL_REQUEST_ID
+```
 
 ## Etymology
 
-[Face control][] in Wikipedia
+[Face control][] in Wikipedia.
+
+[![Sponsored by FunBox](https://funbox.ru/badges/sponsored_by_funbox_centered.svg)](https://funbox.ru)
 
 [Hound]: https://houndci.com
-[Atlassian Stash]: https://www.atlassian.com/software/stash
-[Atlassian Stash Command Line Tools]: https://bitbucket.org/atlassian/stash-command-line-tools
+[Bitbucket Server]: https://www.atlassian.com/software/bitbucket/server
+[Bitbucket Server Command Line Tools]: https://bitbucket.org/atlassian/bitbucket-server-cli
 [RuboCop]: http://batsov.com/rubocop/
 [CoffeeLint]: http://www.coffeelint.org
 [Jenkins]: http://jenkins-ci.org

data/Rakefile

@@ -1,4 +1,3 @@
-require 'bundler/gem_tasks'
 require 'rake/testtask'
 
 Rake::TestTask.new(:test) do |t|

data/face_control.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'face_control'
-  spec.version       = '0.8.1'
+  spec.version       = '0.8.6'
   spec.authors       = ['Ilya Vassilevsky']
   spec.email         = ['vassilevsky@gmail.com']
 
@@ -16,7 +16,6 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'rubocop'
   spec.add_runtime_dependency 'httparty'
 
-  spec.add_development_dependency 'bundler', '~> 1.8'
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'minitest', '~> 5.8'
   spec.add_development_dependency 'minitest-reporters', '~> 1.0'

data/lib/face_control/checkers/comments.rb

@@ -13,7 +13,7 @@ module FaceControl
           file, line_num = line.split(":", 3)
           Comment.new(
             file: file,
-            line: line_num,
+            line: line_num.to_i,
             text: "Do not bury this task in code. Do it now or create a JIRA issue."
           )
         end

data/lib/face_control/checkers/rubocop.rb

@@ -52,21 +52,13 @@ module FaceControl
           text << " — [Guide](#{link})"
         end
 
-        if can_be_autocorrected?(offense)
-          text << " (Run `rubocop -a #{file['path']}` to fix.)"
-        end
-
-        text
+        text << " — #{offense['cop_name']}"
       end
 
       def style_guide_url(offense)
         cop_name = offense['cop_name']
-        cop_config = ::RuboCop::ConfigLoader.default_configuration[cop_name]
-        cop_config['StyleGuide']
-      end
-
-      def can_be_autocorrected?(offense)
-        !offense['corrected'].nil?
+        config = ::RuboCop::ConfigLoader.default_configuration
+        ::RuboCop::Cop::MessageAnnotator.new(config, cop_name, config.for_cop(cop_name), {}).urls.first
       end
     end
   end

data/lib/stash/pull_request/diff.rb

@@ -20,6 +20,7 @@ module Stash
       def diffs_with_added_lines
         @diff['diffs'].select do |diff|
           diff['destination'] &&
+          diff['hunks'] &&
           diff['hunks'].find do |hunk|
             hunk['segments'].find do |segment|
               segment['type'] == 'ADDED'

data/lib/stash/server.rb

@@ -3,6 +3,8 @@ require 'stash/repository'
 
 module Stash
   class Server
+    class CommunicationError < StandardError; end
+
     attr_accessor :root_uri, :user, :password, :logger
 
     def initialize(root_uri, user, password, logger = nil)
@@ -19,14 +21,18 @@ module Stash
     end
 
     def get(path)
-      logged('Response') do
-        HTTParty.get(endpoint + path, auth.merge(logging))
+      with_error_handling do
+        logged('Response') do
+          HTTParty.get(endpoint + path, auth.merge(logging))
+        end
       end
     end
 
     def post(path, data)
-      logged('Response') do
-        HTTParty.post(endpoint + path, auth.merge(content_type).merge(body: data.to_json).merge(logging))
+      with_error_handling do
+        logged('Response') do
+          HTTParty.post(endpoint + path, auth.merge(content_type).merge(body: data.to_json).merge(logging))
+        end
       end
     end
 
@@ -73,5 +79,13 @@ module Stash
         {}
       end
     end
+
+    def with_error_handling
+      yield.tap do |response|
+        unless response.success?
+          fail CommunicationError, 'Stash responded with an error'
+        end
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: face_control
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.8.6
 platform: ruby
 authors:
 - Ilya Vassilevsky
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2016-05-13 00:00:00.000000000 Z
+date: 2020-09-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: docopt
@@ -52,20 +52,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.8'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.8'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -136,7 +122,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
-description: 
+description:
 email:
 - vassilevsky@gmail.com
 executables:
@@ -174,7 +160,7 @@ files:
 homepage: https://github.com/vassilevsky/face_control
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -189,9 +175,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5.1
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Checks Atlassian Stash pull requests and comments on issues in added code
 test_files: []