knapsack_pro 5.2.1 → 5.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ebf6661c02960aefdda92e8fd84ce692e22e59e9fb4d593f677577cda62acb57
-  data.tar.gz: c56e9069badb8ed6090e1b8afb6f2c7d5aebb52a8b7e484484399586b3b7f82c
+  metadata.gz: 81a7fddcd84c41e4e3273ba731e8f68a0063762ebe8234ec8b68cd4cc3e1bcb0
+  data.tar.gz: 213921c7b7d2789b213a80f8a943528f7ae10cf6427c455a04f38ff72a715901
 SHA512:
-  metadata.gz: df2abe40a8b601acd3db8eedce7fc985273e4ff0100346300080ef0de7c8a3df96ecb98f7ab038f121f365d852da0ff877c579a57a652c1c5fb8318a72bc53a7
-  data.tar.gz: 032132b675a088531b3dd112f3258823ec4fd7b9beae8f982dbb3abc85508b4d4ccbd56112ab0e8f1faaf9b0af74edc98fd6896fb01d25ba77f20bef14f61273
+  metadata.gz: 47e7fd9c4c00952b3ebd83dba4de192718100460ad462eb68b90de24dea8df9aa066682771fc3173eaa93c5afa48bd837998711cbebc7dbb911a21d63b3d2cea
+  data.tar.gz: f1b4ac0665596ffe4d83c83c3137f9289a36a709564854155bdf103f38151bcd63b1a58ec165593b0aa2dc3b588b7a497ed032e45ef8c3d5719ee11ee12b747b

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+### 5.3.1
+
+* Avoid noise to stderr when git is not available when collecting authors
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/211
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.3.0...v5.3.1
+
+### 5.3.0
+
+* Perf: Send authors to the API only on the first request (for Queue Mode)
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.2.1...v5.3.0
+
 ### 5.2.1
 
 * Shallow fetch the last month of commits only on CI

data/lib/knapsack_pro/client/api/v1/queues.rb

@@ -17,13 +17,13 @@ module KnapsackPro
                 :node_index => args.fetch(:node_index),
                 :node_build_id => KnapsackPro::Config::Env.ci_node_build_id,
                 :user_seat => KnapsackPro::Config::Env.masked_user_seat,
-                :build_author => KnapsackPro::RepositoryAdapters::GitAdapter.new.build_author,
-                :commit_authors => KnapsackPro::RepositoryAdapters::GitAdapter.new.commit_authors,
               }
 
               if request_hash[:can_initialize_queue] && !request_hash[:attempt_connect_to_queue]
                 request_hash.merge!({
-                  :test_files => args.fetch(:test_files)
+                  :test_files => args.fetch(:test_files),
+                  :build_author => KnapsackPro::RepositoryAdapters::GitAdapter.new.build_author,
+                  :commit_authors => KnapsackPro::RepositoryAdapters::GitAdapter.new.commit_authors,
                 })
               end
 

data/lib/knapsack_pro/repository_adapters/git_adapter.rb

@@ -42,10 +42,10 @@ module KnapsackPro
 
       def git_commit_authors
         if KnapsackPro::Config::Env.ci?
-          `git fetch --shallow-since "one month ago" --quiet`
+          `git fetch --shallow-since "one month ago" --quiet 2>/dev/null`
         end
 
-        `git log --since "one month ago" | git shortlog --summary --email`
+        `git log --since "one month ago" 2>/dev/null | git shortlog --summary --email 2>/dev/null`
       end
 
       def git_build_author

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '5.2.1'
+  VERSION = '5.3.1'
 end

data/spec/knapsack_pro/client/api/v1/queues_spec.rb

@@ -53,6 +53,16 @@ describe KnapsackPro::Client::API::V1::Queues do
         ).and_return(action)
         expect(subject).to eq action
       end
+
+      it "sends authors" do
+        action = double
+
+        expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+          hash_including(request_hash: hash_including(:build_author, :commit_authors))
+        ).and_return(action)
+
+        expect(subject).to eq action
+      end
     end
 
     context 'when can_initialize_queue=false and attempt_connect_to_queue=false' do
@@ -67,15 +77,5 @@ describe KnapsackPro::Client::API::V1::Queues do
         expect(subject).to eq action
       end
     end
-
-    it "sends authors" do
-      action = double
-
-      expect(KnapsackPro::Client::API::Action).to receive(:new).with(
-        hash_including(request_hash: hash_including(:build_author, :commit_authors))
-      ).and_return(action)
-
-      expect(subject).to eq action
-    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 5.2.1
+  version: 5.3.1
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-07-18 00:00:00.000000000 Z
+date: 2023-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake