web_translate_it 2.5.3 → 2.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f4da4e145e768d5dcebc04cb085be79182fa33af33ce4d5fa0b0c1598cef7b4
-  data.tar.gz: d565be12c309530ef0e55b1090804f5e1137b1f327bc08e278477448a4076701
+  metadata.gz: bc4a1374f3018354d15fea035305984fa839a6786526cc88e8d3151f176d8f79
+  data.tar.gz: fbc8868713a8d691169eea12318bb0c288853c25cedf24ddaaac10b44b461d09
 SHA512:
-  metadata.gz: c47cbf387a3557e67ea5d948a2a226d380bb60d3bc9b53755ab858d9ec3f315ae64737d9681c1a3fc4bc5ef29a23a93751ea710dce10576ec2ddfdcc0cd58b47
-  data.tar.gz: 85abf263f0112367bd8f5ae8af8d22e5cb2bdebbb4c31dcc532602b245f389999912bbad8869db0fcf870b7a5e02c7c3000e6fbb9fb89f8221a0ca94eaaeb371
+  metadata.gz: c8ec5eae4ac2941ca048cc798fb4ae5cb5c539345d11286d9aa35a68dca5e35cc3e138cbcb95c0ebf5d8adaffda69938cf41613762612a9ba33836e6f95f2871
+  data.tar.gz: 20af38470f30300f01d260e17d15d16ef90597b72d5e1e5f5d9869cf70a192c65d84f3dc9771ada30264f65c75ebfb0855da9b9f0e94e966cfc908cc872b3eda

data/history.md

@@ -1,3 +1,7 @@
+## Version 2.5.4 / 2022-03-08
+
+* Better support for `before_pull`, `after_pull`, `before_push`, `after_push` hooks. They now display the command output, and write an error message if the exit status of the command was not successful.
+
 ## Version 2.5.3 / 2022-02-04
 
 * Properly fix ruby compatibility issues.

data/lib/web_translate_it/command_line.rb

@@ -36,7 +36,7 @@ module WebTranslateIt
     def pull
       complete_success = true
       STDOUT.sync = true
-      `#{configuration.before_pull}` if configuration.before_pull
+      before_pull_hook
       # Selecting files to pull
       files = []
       fetch_locales_to_pull.each do |locale|
@@ -70,15 +70,37 @@ module WebTranslateIt
         threads.each { |thread| thread.join }
         time = Time.now - time
         puts "Pulled #{files.size} files at #{(files.size/time).round} files/sec, using #{n_threads} threads."
-        `#{configuration.after_pull}` if configuration.after_pull
+        after_pull_hook
         complete_success
       end
     end
-    
+
+    def before_pull_hook
+      if configuration.before_pull
+        output = `#{configuration.before_pull}`
+        if $?.success?
+          puts output
+        else
+          abort 'Error: exit code for before_pull command is not zero'
+        end
+      end
+    end
+
+    def after_pull_hook
+      if configuration.after_pull
+        output = `#{configuration.after_pull}`
+        if $?.success?
+          puts output
+        else
+          abort 'Error: exit code for after_pull command is not zero'
+        end
+      end
+    end
+
     def push
       complete_success = true
       STDOUT.sync = true
-      `#{configuration.before_push}` if configuration.before_push
+      before_push_hook
       WebTranslateIt::Connection.new(configuration.api_key) do |http|
         fetch_locales_to_push(configuration).each do |locale|
           if parameters.any?
@@ -96,9 +118,31 @@ module WebTranslateIt
           end
         end
       end
-      `#{configuration.after_push}` if configuration.after_push
+      after_push_hook
       complete_success
     end
+
+    def before_push_hook
+      if configuration.before_push
+        output = `#{configuration.before_push}`
+        if $?.success?
+          puts output
+        else
+          abort 'Error: exit code for before_push command is not zero'
+        end
+      end
+    end
+
+    def after_push_hook
+      if configuration.after_push
+        output = `#{configuration.after_push}`
+        if $?.success?
+          puts output
+        else
+          abort 'Error: exit code for after_push command is not zero'
+        end
+      end
+    end
     
     def add
       complete_success = true

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: web_translate_it
 version: !ruby/object:Gem::Version
-  version: 2.5.3
+  version: 2.5.4
 platform: ruby
 authors:
 - Edouard Briere
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-04 00:00:00.000000000 Z
+date: 2022-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multipart-post
@@ -155,7 +155,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.9
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: A CLI to sync locale files with WebTranslateIt.com.