gitarro 0.1.83 → 0.1.84

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b86f19aa072c0fc665edacfec17e6d83bb2d73abf2f18a905ac22b34087dfc3
-  data.tar.gz: 813daa1130b446fb366c82feb9ac3de40f4ad45709ec7601eeccf5b4e1bcf6e5
+  metadata.gz: 76a009c633d69576084ebb47ccd9162935371adde4fdc6345cd5c170db43bf16
+  data.tar.gz: b8111a0583e8ce8f379f34f511e8979d68e2d21eca2f369fc93d2a6c003fe1c7
 SHA512:
-  metadata.gz: 9a3e56d40cc0bcaf2dd90927ede914751b3f037e68fc4612363967981c4eddc17bda3add0094a6b4a13c9e0daf6689a072c69b74f146e87a77c364b34a2a6c49
-  data.tar.gz: 6148b5ee02c96185871f3588d25d0e7ede02e42d213e4a2a67edcf3a90cb611356b6d4ed48e5eeb8780cfa180565d62b5a80ffc7931c6275c3d8839065a6bb15
+  metadata.gz: 76275e058bb4b7a33f2cac0fc32bd568e047b3bd79fdff2ead753a5987d0bfc795ba38619780bf7d55364ccbf4899612c7a04320dc7abc95acd6279afb94ceea
+  data.tar.gz: ba53a4161351cf6b1a88716d95f0296df056e1daafa275dd14d7d6d5059a4bf432c0cf67e030a89175f2cdcda2179b42a4671385d89bd3e66d2cc19bd4c80a5f

data/bin/gitarro

@@ -8,6 +8,8 @@ require 'gitarro/git_op'
 require 'gitarro/backend'
 
 b = Backend.new
+exit 0 if b.triggered_by_pr_number?
+
 prs = b.open_newer_prs
 exit 0 if prs.empty?
 
@@ -18,8 +20,6 @@ prs.each do |pr|
 
   # this check the last commit state, catch for review or not reviewd status.
   comm_st = b.client.status(b.repo, pr.head.sha)
-  # pr number trigger.
-  break if b.triggered_by_pr_number?
 
   # retrigger if magic word found
   b.retrigger_check(pr)

data/lib/gitarro/backend.rb

@@ -196,7 +196,6 @@ class Backend
   def retrigger_check(pr)
     return unless retrigger_needed?(pr)
 
-    create_status(pr, 'pending')
     print_test_required
     gbexec.export_pr_data(pr)
     exit 0 if @check
@@ -217,7 +216,6 @@ class Backend
   def unreviewed_new_pr?(pr, comm_st)
     return unless commit_is_unreviewed?(comm_st)
 
-    pr_all_files_type(pr.number, @file_type)
     return if empty_files_changed_by_pr?(pr)
 
     # gb.check is true when there is a job running as scheduler
@@ -263,9 +261,18 @@ class Backend
   def retriggered_by_checkbox?(pr, context)
     return false unless pr.body.include? "[x] Re-run test \"#{context}\""
 
-    puts "Re-run test \"#{context}\""
+    return true if @check
+
+    skipped = ''
+    unless empty_files_changed_by_pr?(pr)
+      skipped = '(Test skipped, there are no changes to test)'
+    end
+
+    puts "Re-run test \"#{context}\" #{skipped}"
     new_pr_body = pr.body.gsub("[x] Re-run test \"#{context}\"",
-                               "[ ] Re-run test \"#{context}\"")
+                               "[ ] Re-run test \"#{context}\" #{skipped}")
+    new_pr_body = new_pr_body.gsub("#{skipped} #{skipped}", skipped) unless skipped.empty?
+                    
     @client.update_pull_request(@repo, pr.number, body: new_pr_body)
     true
   end
@@ -339,6 +346,6 @@ class Backend
 
     # if check is set, the comment in the trigger job will be del.
     # so setting it to pending, it will be remembered
-    pr_all_files_type(pr.number, @file_type).any?
+    empty_files_changed_by_pr?(pr)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitarro
 version: !ruby/object:Gem::Version
-  version: 0.1.83
+  version: 0.1.84
 platform: ruby
 authors:
 - Dario Maiocchi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-02-20 00:00:00.000000000 Z
+date: 2019-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: english