drone-builds 0.2.2 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3dc8566a96dc4b85497b5796023fb0346e909a8e
-  data.tar.gz: 2c088fd00f33c43c6dae547f2838d8c973e70874
+  metadata.gz: ebc202d5e431628b68d6b18139e09ab8017e3611
+  data.tar.gz: 14748c8228a690c14644404a517737845886f633
 SHA512:
-  metadata.gz: b123a6b8cd76bd821c7a2158710f02ea008d42cd6724f4686df28ec4b4522f01eed743a6709dc1a45d6d05a7aac95964cf7e347f85151468344b42fdce54c84d
-  data.tar.gz: 12cec74ae0d997d74c51d35440d5b836346fb0d10a369d16d0690cc712e44493fdecbcdc4c4a286cc49d0ce9e1e2c21da36eaa21a2904d6fab88b71dfadeecca
+  metadata.gz: f6ad38567302652d0e020b9de47da175ff04003d0413be8438004d5a696abab90674e54a7efb1ccff6c94660d3d494d96329e9e483ee2557835978b50adf3ced
+  data.tar.gz: 0eed21a1d304959e6a9f55fa27d432f715ee5b87fbcc3db67754b913afc0d7c9ee1f9bae85aef10682e6e257d16ff1ef092be8e80a15ef0a4910b24ae0e786b7

data/Rakefile

@@ -1,5 +1,5 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 

data/bin/console

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
-require "bundler/setup"
-require "drone/builds"
+require 'bundler/setup'
+require 'drone/builds'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -10,5 +10,5 @@ require "drone/builds"
 # require "pry"
 # Pry.start
 
-require "irb"
+require 'irb'
 IRB.start(__FILE__)

data/drone-builds.gemspec

@@ -5,29 +5,29 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'drone/builds/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "drone-builds"
+  spec.name          = 'drone-builds'
   spec.version       = Drone::Builds::VERSION
-  spec.authors       = ["Dorian Marié"]
-  spec.email         = ["dorian@doma.io"]
+  spec.authors       = ['Dorian Marié']
+  spec.email         = ['dorian@doma.io']
 
-  spec.summary       = %q{Command line interface to Drone}
-  spec.description   = %q{List builds, print realtime output, view previous outputs, filter outputs}
-  spec.homepage      = "https://github.com/Dorian/drone-builds"
-  spec.license       = "MIT"
+  spec.summary       = 'Command line interface to Drone'
+  spec.description   = 'List builds, print realtime output, view previous outputs, filter outputs'
+  spec.homepage      = 'https://github.com/Dorian/drone-builds'
+  spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency "colorize", "~> 0"
-  spec.add_runtime_dependency "home-dotenv", "~> 0"
-  spec.add_runtime_dependency "json", "~> 2"
-  spec.add_runtime_dependency "terminal-table", "~> 1"
-  spec.add_runtime_dependency "thor", "~> 0"
-  spec.add_development_dependency "bundler", "~> 1"
-  spec.add_development_dependency "rake", "~> 12"
-  spec.add_development_dependency "rspec", "~> 3"
+  spec.add_runtime_dependency 'colorize', '~> 0'
+  spec.add_runtime_dependency 'home-dotenv', '~> 0'
+  spec.add_runtime_dependency 'json', '~> 2'
+  spec.add_runtime_dependency 'terminal-table', '~> 1'
+  spec.add_runtime_dependency 'thor', '~> 0'
+  spec.add_development_dependency 'bundler', '~> 1'
+  spec.add_development_dependency 'rake', '~> 12'
+  spec.add_development_dependency 'rspec', '~> 3'
 end

data/exe/drone-builds

@@ -4,4 +4,3 @@ require 'drone/builds'
 require 'drone/builds/cli'
 
 Drone::Builds::Cli.start
-

data/lib/drone/builds.rb

@@ -1,4 +1,4 @@
-require "drone/builds/version"
+require 'drone/builds/version'
 require 'open-uri'
 require 'net/http'
 require 'json'
@@ -11,7 +11,6 @@ module Drone
     DEFAULT_JOB_ID = 1
 
     def initialize(token:, domain:, owner:, name:, job_id: nil, protocol: nil)
-      @protocol
       @token = token
       @domain = domain
       @owner = owner
@@ -60,13 +59,9 @@ module Drone
       uri = URI(full_url(url))
 
       Net::HTTP.start(uri.host, uri.port) do |http|
-        request = Net::HTTP::Get.new uri
-
-        http.request request do |response|
+        http.request(Net::HTTP::Get.new(uri)) do |response|
           response.read_body do |chunk|
-            lines = chunk.lines.grep(/^data:/).map { |l| l.gsub(/^data:/, '') }
-            output = lines.map(&:strip).compact.join("\n")
-            yield output
+            yield clean_stream_data(chunk)
           end
         end
       end
@@ -80,4 +75,9 @@ module Drone
       open(full_url(url)).read
     end
   end
+
+  def clean_stream_data(data)
+    lines = data.lines.grep(/^data:/).map { |l| l.gsub(/^data:/, '') }
+    lines.map(&:strip).compact.join("\n")
+  end
 end

data/lib/drone/builds/cli.rb

@@ -2,35 +2,36 @@ require 'home-dotenv/load'
 require 'thor'
 require 'drone/builds/parser'
 
+# Command-line interface configuration
 class Drone::Builds::Cli < Thor
-  package_name "drone-builds"
+  package_name 'drone-builds'
 
-  map "l" => :list
-  map "s" => :show
+  map 'l' => :list
+  map 's' => :show
 
-  desc "list", "list last builds"
+  desc 'list', 'list last builds'
   method_option :limit,
-    type: :numeric,
-    default: Drone::Builds::DEFAULT_LIMIT,
-    aliases: "-l",
-    desc: "number of builds shown"
+                type: :numeric,
+                default: Drone::Builds::DEFAULT_LIMIT,
+                aliases: '-l',
+                desc: 'number of builds shown'
   def list
     puts self.class.parser.table(
       self.class.parser.parse_builds(self.class.client.list(limit: options[:limit])),
-      %i(id status ref message)
+      %i[id status ref message]
     )
   end
 
-  desc "show", "show a build"
-  method_option :filter, type: :string, aliases: "-f", desc: "filter builds output lines"
-  method_option :last, type: :numeric, default: 1, aliases: "-l", desc: "show nth last build"
+  desc 'show', 'show a build'
+  method_option :filter, type: :string, aliases: '-f', desc: 'filter builds output lines'
+  method_option :last, type: :numeric, default: 1, aliases: '-l', desc: 'show nth last build'
   def show(id = nil)
-    if id.nil? && options[:last]
+    if id.nil? || id == 'l' || id == 'last'
       last = self.class.client.list(limit: options[:last])[0]
-      id = last["id"] if last
+      id = last['id'] if last
     end
 
-    abort "id is required" if id.to_s.empty?
+    abort 'id is required' if id.to_s.empty?
 
     self.class.client.show(id, filter: options[:filter]) { |line| puts line }
   end

data/lib/drone/builds/parser.rb

@@ -10,7 +10,7 @@ class Drone::Builds::Parser
     failure: :red,
     killed: :white,
     pending: :yellow
-  }
+  }.freeze
 
   def initialize(client)
     @client = client
@@ -22,10 +22,10 @@ class Drone::Builds::Parser
 
   def parse_build(build)
     {
-      id: parse_id(build["id"]),
-      status: parse_status(build["status"]),
-      ref: parse_ref(build["ref"]),
-      message: parse_message(build["message"])
+      id: parse_id(build['id']),
+      status: parse_status(build['status']),
+      ref: parse_ref(build['ref']),
+      message: parse_message(build['message'])
     }
   end
 
@@ -38,11 +38,11 @@ class Drone::Builds::Parser
   end
 
   def parse_ref(ref)
-    ref.gsub("refs/heads/", "").gsub("/head", "").gsub("refs/pull/", '#')
+    ref.gsub('refs/heads/', '').gsub('/head', '').gsub('refs/pull/', '#')
   end
 
   def parse_message(message)
-    message.to_s.lines.first.to_s.gsub("of github.com:#{client.owner}/#{client.name}", "")[0..50]
+    message.to_s.lines.first.to_s.gsub("of github.com:#{client.owner}/#{client.name}", '')[0..50]
   end
 
   def table(data, fields)

data/lib/drone/builds/version.rb

@@ -1,5 +1,5 @@
 module Drone
   class Builds
-    VERSION = "0.2.2"
+    VERSION = '0.2.3'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: drone-builds
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
 platform: ruby
 authors:
 - Dorian Marié