face_control 0.8.5 → 0.8.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c3d2f48370bafab2fa06775866f106d9a03dfe8f
-  data.tar.gz: 4e60c9742597486b72e946036db07aaf10714ae4
+SHA256:
+  metadata.gz: 325aba4c4fdce156c0dae24d9c5cf10b2993b4b70e0fcf184d3645cea22f5563
+  data.tar.gz: 700af54ea35e67f76d9245acb0fc5e63dfaa766295b357d27d71cf5e0a7de3a2
 SHA512:
-  metadata.gz: c915bd12dd864727f4cf80b72c235fa8b29cceba254e09671416b272ab481c389613a2e0f0bcc461f24fc1b99782e45a3e15303c35cdb5dff764107ff230956c
-  data.tar.gz: d716c1f70a7c173c370c39c8abdcea26be832a54b20bcf45977a6336f180adcbc7848e507644fedbb4906d24affced837c5372ea7c3ef6d62e4b6f595cc6e5de
+  metadata.gz: d5df85b4788e83c52f3cb051fee19d5137794b3ae9096e2bcc626b0585783515bb1479d52bf9c1cc2fab1d0cd829cbbc69120ccdf48f95e9e686144e4a047163
+  data.tar.gz: 308cb4a0b3d91dc19b1da72b537ed7dbd17f327554bbbc091e7ef16cd895f4828391490843ae3e63c554e0c73d4ce7935bb86131120d176b094f2390a70c9f0f

data/.gitignore

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

data/README.md

@@ -1,11 +1,11 @@
+# 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.
 
@@ -16,61 +16,75 @@ Inspired by [Hound][].
 
 ## Installation
 
-    gem install face_control
+```bash
+gem install face_control
+```
 
-You also need to have CoffeeLint installed and available in PATH.
+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).
 
-It's natural to run this on a continuous integration server.
-For example, here's a [Jenkins][] project setup:
+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>
+```
 
-* Source Code Management
-  * Git
-    * Repositories
-      * Refspec:
+Instead of `--skip-severity` you can use just `-S`.
+
+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 [Bitbucket Server 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
 [Bitbucket Server]: https://www.atlassian.com/software/bitbucket/server

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.5'
+  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/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']
         config = ::RuboCop::ConfigLoader.default_configuration
-        ::RuboCop::Cop::MessageAnnotator.new(config, config.for_cop(cop_name), {}).urls.first
-      end
-
-      def can_be_autocorrected?(offense)
-        !offense['corrected'].nil?
+        ::RuboCop::Cop::MessageAnnotator.new(config, cop_name, config.for_cop(cop_name), {}).urls.first
       end
     end
   end

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.5
+  version: 0.8.6
 platform: ruby
 authors:
 - Ilya Vassilevsky
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2017-04-03 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.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: []