drone-builds 0.6.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a9bb029084c1280e627a9df2bed06883e3239d94
-  data.tar.gz: 99b0caf59b673b2b181cc63c9cf356e073c26b67
+  metadata.gz: 22620afbf57da75092a7eceffeb7d6c1c2834e81
+  data.tar.gz: ce1c8d7baa86f5959f34899dc38630b5f2aefc27
 SHA512:
-  metadata.gz: 5fb6200e2f06aaa53732eb7bc7e6b2fabb5f2e4305a158509e33c03feec5cd6dc2b1647735c316e9e9f0f947a89d6c3402fa0666e34d29681b22eabab282803f
-  data.tar.gz: d668212c58243294a317a82f4b17b92598ff1b8cdca84b6ca1ca4ff410cdd85060692fa57f6b159701a67b46d5005dab3b27f74be26ac69b9968d42054fe48c6
+  metadata.gz: 24f3d33ff66baa14a5469f26eb9d319d9cbf5b149cbecaabeba31fcaa1ac5cb2a9e19534a7fa3bed4313072dd592b3327f6e88cf179720fbf8128fbf14ff7f70
+  data.tar.gz: 6e9ff1c3284faeeaf408761bbcd40a47ad1430254c7fa89211be1d76442abde6ba1865e023ce0f4c5af993fd1aea5dc062cba6d249918b8dc1fb1dcbeec0708d

data/lib/drone/builds.rb

@@ -26,22 +26,22 @@ module Drone
       end.first(limit || DEFAULT_LIMIT).reverse
     end
 
-    def show(id, filter: nil, context: nil)
-      text_api(logs_url(id)).lines.each do |l|
+    def show(build_number, filter: nil, context: nil)
+      text_api(logs_url(build_number)).lines.each do |l|
         yield l if show_line(l, filter: filter, context: context)
       end
     rescue OpenURI::HTTPError
-      stream_api(stream_url(id)) do |l|
+      stream_api(stream_url(build_number)) do |l|
         yield l if show_line(l, filter: filter, context: context)
       end
     end
 
-    def kill(id)
-      delete_api(job_url(id))
+    def kill(build_number)
+      delete_api(job_url(build_number))
     end
 
-    def restart(id)
-      post_api(build_url(id))
+    def restart(build_number)
+      post_api(build_url(build_number))
     end
 
     private
@@ -54,20 +54,20 @@ module Drone
       "#{base_url}/repos/#{owner}/#{name}/builds"
     end
 
-    def logs_url(id)
-      "#{base_url}/repos/#{owner}/#{name}/logs/#{id}/#{job_id}"
+    def logs_url(build_number)
+      "#{base_url}/repos/#{owner}/#{name}/logs/#{build_number}/#{job_id}"
     end
 
-    def build_url(id)
-      "#{base_url}/repos/#{owner}/#{name}/builds/#{id}"
+    def build_url(build_number)
+      "#{base_url}/repos/#{owner}/#{name}/builds/#{build_number}"
     end
 
-    def job_url(id)
-      "#{base_url}/repos/#{owner}/#{name}/builds/#{id}/#{job_id}"
+    def job_url(build_number)
+      "#{base_url}/repos/#{owner}/#{name}/builds/#{build_number}/#{job_id}"
     end
 
-    def stream_url(id)
-      "#{base_url}/stream/#{owner}/#{name}/#{id}/#{job_id}"
+    def stream_url(build_number)
+      "#{base_url}/stream/#{owner}/#{name}/#{build_number}/#{job_id}"
     end
 
     def access_token_param

data/lib/drone/builds/cli.rb

@@ -28,7 +28,7 @@ class Drone::Builds::Cli < Thor
       status: options[:status]
     ))
 
-    puts self.class.parser.table(builds, %i(id status author ref message))
+    puts self.class.parser.table(builds, %i(number status author ref message))
   end
 
   desc 'show', 'show a build'
@@ -38,32 +38,35 @@ class Drone::Builds::Cli < Thor
   method_option :pull, type: :numeric, aliases: '-p', desc: 'show last build from pull request ID'
   method_option :branch, type: :string, aliases: '-b', desc: 'builds on that branch'
   method_option :status, type: :string, aliases: '-s', desc: 'success / failure / running / killed'
-  def show(id = nil)
+  def show(build_number = nil)
     ref = self.class.options_to_ref(options)
 
-    if id.to_i.to_s != id
-      id = self.class.client.list(
+    if build_number.to_i.to_s != build_number
+      build_number = self.class.client.list(
         limit: options[:last]&.abs,
         ref: ref,
         status: options[:status]
-      )[0]&.[]('id')
+      )[0]&.[]('build_number')
     end
 
-    abort "No build found for options: #{options}" unless id
+    abort "No build found for options: #{options}" unless build_number
 
-    self.class.client.show(id, filter: options[:filter], context: options[:context]) do |line|
+    self.class.client.show(
+      build_number,
+      filter: options[:filter],
+      context: options[:context]) do |line|
       puts line
     end
   end
 
   desc 'restart', 'restart a build'
-  def restart(id = nil)
-    puts self.class.client.restart(id)
+  def restart(build_number = nil)
+    puts self.class.client.restart(build_number)
   end
 
   desc 'kill', 'kill a build'
-  def kill(id = nil)
-    puts self.class.client.kill(id)
+  def kill(build_number = nil)
+    puts self.class.client.kill(build_number)
   end
 
   desc 'version', 'version used'

data/lib/drone/builds/parser.rb

@@ -24,7 +24,7 @@ class Drone::Builds::Parser
 
   def parse_build(build)
     {
-      id: parse_id(build['id']),
+      number: parse_number(build['number']),
       status: parse_status(build['status']),
       author: parse_author(build['author']),
       ref: parse_ref(build['ref']),
@@ -32,8 +32,8 @@ class Drone::Builds::Parser
     }
   end
 
-  def parse_id(id)
-    id.to_s
+  def parse_number(number)
+    number.to_s
   end
 
   def parse_status(status)

data/lib/drone/builds/version.rb

@@ -1,5 +1,5 @@
 module Drone
   class Builds
-    VERSION = '0.6.1'.freeze
+    VERSION = '0.7.0'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: drone-builds
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.7.0
 platform: ruby
 authors:
 - Dorian Marié
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-05-13 00:00:00.000000000 Z
+date: 2017-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize