safe_pusher 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8dd851e29beb0acaa544597d36c9ba03c6ef4620
-  data.tar.gz: b87276b61f21dfe2ec419ac90f7eaff97e7eb89d
+  metadata.gz: 6f14a1f7069630411c76a8ec0c1a33f75f3b0d95
+  data.tar.gz: c5b212741108b33951374c339fa888aa1215c94f
 SHA512:
-  metadata.gz: ce3fff993eaffda06bcc60c6a3a71f4ff798496157b9b350201af19a9fea34296fa419f6b257c1313709cd831885b1993c34b073be00a093174fb0859b20b326
-  data.tar.gz: 2c8c6c017755eebe779459c981eceddaed229ea7c3338f03a91d79684b138e20d85942e05a35066e193f1d9ccebb895c854c50b44d81a1bcf0b84c435572d227
+  metadata.gz: dc2fa8785c5401f0b47db636e874a4cc9a3b0ed3355633ab3c7aa89df5ddd2228e7bd76177bf85a0fdcab5a79003c956ac3099d4b255bdcd0eafde3284e6ea83
+  data.tar.gz: '02567809eba97e24af3b0ee8fac29bc9b407696ac07896cf763d3527582e191629d1a0580e0371377166bf7bda0502d9fb9c6f3f57463bf65faf0c832db06226'

data/.travis.yml

@@ -1,8 +1,18 @@
 ---
+env:
+  global:
+    - CC_TEST_REPORTER_ID=5e03a69363f26aba0c9c779a841db044295bf61762f9b7d719e2c064497f24e0
 sudo: false
 language: ruby
 cache: bundler
 rvm:
   - 2.3.7
 before_install: gem install bundler -v 1.16.3
+
+before_script:
+  - curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
+  - chmod +x ./cc-test-reporter
+  - ./cc-test-reporter before-build
 script: bundle exec rspec spec
+after_script:
+  - ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 This project adheres to [Semantic Versioning](http://semver.org)
 
+## [0.2.2] - 2019-01-05
+Improvements:
+ - Removing thor
+ - Using custom CLI to handle multiple commands
+
 ## [0.2.1] - 2018-12-31
 Improvement:
  - Deprecating rainbow to install pronto and pronto Rubocop

data/README.md

@@ -1,15 +1,18 @@
-# SafePusher
-
 [![Build Status](https://travis-ci.com/williampollet/safe_pusher.svg?branch=master)](https://travis-ci.com/williampollet/safe_pusher)
+[![Maintainability](https://api.codeclimate.com/v1/badges/1aa6c275f9ce4d4c6ec3/maintainability)](https://codeclimate.com/github/williampollet/safe_pusher/maintainability)
+[![Test Coverage](https://api.codeclimate.com/v1/badges/1aa6c275f9ce4d4c6ec3/test_coverage)](https://codeclimate.com/github/williampollet/safe_pusher/test_coverage)
 
-Run your favorite linters and specs on the files you touched, before pushing your branch
+# SafePusher
+
+Run your favorite linters and specs on the files you touched, before pushing your branch.
 
 ## Installation
 
-Add this line to your application's Gemfile:
+Add these lines to your application's Gemfile:
 
 ```ruby
-gem 'safe_pusher'
+# Lint and launch specs before pushing.
+gem 'safe_pusher', require: false
 ```
 
 And then execute:
@@ -20,17 +23,15 @@ Or install it yourself as:
 
     $ gem install safe_pusher
 
-
 To use the gem fully, you should install [pronto](https://github.com/prontolabs/pronto) and [rspec](https://github.com/rspec/rspec) first.
 
 ## Configuration
 
-Generate the `binstubs` for `pronto` and `rspec`:
+Generate the `binstubs` for `pronto`, `rspec` and `safepush`:
 
-    $ bundler binstubs pronto
-    $ bundler binstubs rspec-core
+    $ bundler binstubs pronto rspec-core safe_pusher
 
-create the `safe_pusher.yml` file at the root of your application
+Create the `safe_pusher.yml` file at the root of your application:
 
 ```yaml
 files_to_skip:
@@ -46,13 +47,21 @@ To see the commands available, type:
 
     $ safepush
 
-To run pronto checks before to push on github run:
+To run pronto checks before you push to GitHub run:
+
+    $ safepush lint_push_and_open
+
+or
+
+    $ safepush lpo
+
+To run specs and pronto before you push to GitHub run:
 
-    $ safepush ppush
+    $ safepush test_lint_push_and_open
 
-To run specs and pronto before to push on github run:
+or
 
-    $ safepush ppushtest
+    $ safepush tlpo
 
 ## Contributing
 

data/exe/safepush

@@ -1,4 +1,4 @@
 #!/usr/bin/env ruby
 require 'safe_pusher'
 
-SafePusher::CLI.start
+SafePusher::CLI.new(arguments: ARGV).start

data/lib/safe_pusher/cli.rb

@@ -1,18 +1,47 @@
 module SafePusher
-  class CLI < Thor
-    desc 'test (t)', 'launch the test suite'
+  class CLI
+    attr_reader :arguments
+
+    def initialize(arguments:)
+      @arguments = arguments
+    end
+
+    def start
+      unless arguments_valid?
+        help
+
+        return
+      end
+
+      arguments.each do |command|
+        execute_command(command)
+      end
+    end
+
+    private
+
+    def execute_command(command)
+      case command
+      when 'test', 't'
+        test
+      when 'lint', 'l'
+        lint
+      when 'open', 'o'
+        open
+      when 'push', 'p'
+        push
+      end
+    end
+
     def test
       puts '##########################'.yellow
       puts '## Testing new files... ##'.yellow
       puts '##########################'.yellow
-
       results = SafePusher::RspecRunner.new.call
 
       exit results unless results == 0
     end
-    map 't' => :test
 
-    desc 'lint (l)', 'launch the linters'
     def lint
       puts '#######################'.yellow
       puts '## Running linter... ##'.yellow
@@ -22,9 +51,7 @@ module SafePusher
 
       exit results unless results == 0
     end
-    map 'l' => :lint
 
-    desc 'push (p)', 'push your code on github'
     def push
       puts '##########################'.yellow
       puts '## Pushing to Github... ##'.yellow
@@ -34,9 +61,7 @@ module SafePusher
 
       exit results unless results == 0
     end
-    map 'p' => :push
 
-    desc 'open (o)', 'open a pull request on github'
     def open
       puts '#########################################'.yellow
       puts '## Opening a pull request on Github... ##'.yellow
@@ -46,46 +71,21 @@ module SafePusher
 
       exit results unless results == 0
     end
-    map 'o' => :open
-
-    desc 'push_and_open (po)', 'push your code on github,'\
-         ' and open a Pull Request on Github if none is openned'
-    def push_and_open
-      puts '##########################'.yellow
-      puts '## Pushing to Github... ##'.yellow
-      puts '##########################'.yellow
 
-      results = SafePusher::GithubRunner.new.push_and_open
-
-      exit results unless results == 0
+    def arguments_valid?
+      arguments.join(' ') =~ /^(?!\s*$)(?:test|lint|push|open|t|l|p|o| )+$/
     end
-    map 'po' => :push_and_open
 
-    desc 'test_and_lint (tl)',
-         'launch the test suite, then the linters if it is successful'
-    def test_and_lint
-      invoke :test
-      invoke :lint
-    end
-    map 'tl' => :test_and_lint
-
-    desc 'lint_push_and_open (lpo)',
-         'lanch the linters, then push on github and open a'\
-         ' Pull Request on Github if none is openned'
-    def lint_push_and_open
-      invoke :lint
-      invoke :push_and_open
-    end
-    map 'lpo' => :lint_push_and_open
-
-    desc 'test_lint_push_and_open (tlpo)',
-         'lanch the linters, launch test suite, then push on github and open'\
-         'a Pull Request if none is openned'
-    def test_lint_push_and_open
-      invoke :test
-      invoke :lint
-      invoke :push_and_open
+    def help
+      puts "Usage:\n"\
+      " help (h) # show this usage message\n"\
+      " ##########################################################\n"\
+      " # you can use any combination of theese commands \n"\
+      " ##########################################################\n"\
+      " test (t) # run the test suite\n"\
+      " lint (l) # run the linters\n"\
+      " push (p) # push on distant repository\n"\
+      ' open (o) # open a pull request on the distant repository'
     end
-    map 'tlpo' => :test_lint_push_and_open
   end
 end

data/lib/safe_pusher/github_runner.rb

@@ -3,8 +3,8 @@ require 'English'
 
 module SafePusher
   class GithubRunner
-    def push_and_open
-      push
+    def push
+      system('git push origin')
 
       exit_status = $CHILD_STATUS.exitstatus
 
@@ -14,17 +14,11 @@ module SafePusher
         push_and_set_upstream
 
         exit_status = $CHILD_STATUS.exitstatus
-
-        open if exit_status == 0
       end
 
       exit_status
     end
 
-    def push
-      system('git push origin')
-    end
-
     def open
       system(
         "open '#{SafePusher.configuration.repo_url}/pull/new/#{branch}'",

data/lib/safe_pusher/version.rb

@@ -1,3 +1,3 @@
 module SafePusher
-  VERSION = '0.2.1'.freeze
+  VERSION = '0.2.2'.freeze
 end

data/safe_pusher.gemspec

@@ -37,9 +37,8 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'colorize', '~> 0.8.1'
-  spec.add_dependency 'thor', '~> 0.19.4'
 
-  spec.add_development_dependency 'bundler', '~> 1.16'
+  spec.add_development_dependency 'bundler', '>= 1.16'
   spec.add_development_dependency 'fashion_police', '~> 1.2'
   spec.add_development_dependency 'pronto', '~> 0.9.5'
   spec.add_development_dependency 'pronto-rubocop', '~> 0.9.1'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: safe_pusher
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - William Pollet
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-12-31 00:00:00.000000000 Z
+date: 2019-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
@@ -24,32 +24,18 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.8.1
-- !ruby/object:Gem::Dependency
-  name: thor
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.19.4
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.19.4
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.16'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.16'
 - !ruby/object:Gem::Dependency