captive-release 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '028b75d95da7f0c5ae533774c1fc1bc353234bd569a58f9c83798e0519a592c2'
-  data.tar.gz: bb5a5ffbbaf260dad04cd28afb303c476ae5dff295454bbf1b95708bfba58126
+  metadata.gz: a9e947f971c93535758990c20c7684e5178c345bfee6abddef43ee6584b1378a
+  data.tar.gz: bc5c688c4d49ad3326104a42e4209e9b99f22cd3dd3511cf5761c8843465053c
 SHA512:
-  metadata.gz: ee228007f5b5cb48c7d2fb2402a019c287b4d2a8dc1e37df0e6a0f3c159af97ce4aa3e5e95e57d6b1b947538414c081ea78a998a056a8db80770d263ea3371d9
-  data.tar.gz: 1a0ae25c85c204a1ac5431a4d1e704b67715b77c281763b103ff90da04a1a8550b628f4f1e3a785d743a1ec78b9c2d02b328ef1bee37ec5955085ad12b59666d
+  metadata.gz: c289101fc72ca8463baad9fd904c060c6a67890d2b0ece8c61e44ddca8df0d2ff387487b67facfcf1c16e69d90ac61516b83728ef1025ad8d8894bea76fbd680
+  data.tar.gz: 00bf4e5aad63a7ce24786c6a4c39aa1481244ec8bdf5725595084e49c783f98f66d5222a2f39b7adc149d553c931471dd23457fb49b6cf278df64bbf7f3cbc94

data/lib/captive_release/exclusion_input.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module CaptiveRelease
+  # Ce que veut dire la saisie d'exclusion : des rangs séparés par des virgules, numérotés à
+  # partir de 1 comme la liste affichée, et vide pour tout garder.
+  #
+  # Cette unité change quand la syntaxe de la saisie change, VersionFileSelection quand la
+  # façon de présenter le choix change.
+  module ExclusionInput
+    def self.apply(found, saisie)
+      return found if saisie.strip.empty?
+
+      rangs = saisie.split(",").map { |numero| numero.strip.to_i - 1 }
+      found.reject.with_index { |_, rang| rangs.include?(rang) }
+    end
+  end
+end

data/lib/captive_release/gemfile_lock_refresh.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
 module CaptiveRelease
-  # Remet Gemfile.lock d'aplomb après un bump : un .gemspec à la racine porte la version
-  # bumpée, donc Gemfile.lock devient périmé et --deployment le refuse en CI si personne ne
-  # le relance.
+  # Remet les Gemfile.lock d'aplomb après un bump : un .gemspec porte la version bumpée, donc
+  # le Gemfile.lock de son dossier devient périmé et --deployment le refuse en CI si personne
+  # ne le relance. Dans un monorepo, ce dossier n'est pas forcément la racine.
   #
-  # Cette unité change quand la façon de détecter ou de relancer le lock change,
+  # Cette unité change quand la façon de détecter ou de relancer les locks change,
   # ReleaseCommand quand la suite des étapes change.
   class GemfileLockRefresh
     LOCKFILE = "Gemfile.lock"
@@ -14,17 +14,28 @@ module CaptiveRelease
       @files = files
     end
 
+    # Les Gemfile.lock à relancer, relatifs à la racine : un par dossier qui porte un .gemspec.
+    #
     # rescue : une arborescence hostile ne doit jamais faire échouer la release, la détection
     # n'est qu'une commodité. Une fois vraie, l'exécution du bundle lock, elle, échoue haut
     # la main si elle rate.
-    def needed?
-      File.exist?(@files.path(LOCKFILE)) && @files.glob("*.gemspec").any? && !GitCommands.ignored?(LOCKFILE)
+    def lockfiles
+      @files.glob("**", LOCKFILE).map { |lock| @files.relative(lock) }.select { |lock| a_relancer?(lock) }
     rescue StandardError
-      false
+      []
     end
 
-    def call
-      Shell.run! "bundle lock"
+    def call(locks = lockfiles)
+      locks.each { |lock| Shell.run! commande(lock) }
     end
+
+    private
+
+    def a_relancer?(lock)
+      @files.glob(File.dirname(lock), "*.gemspec").any? && !GitCommands.ignored?(lock)
+    end
+
+    # Le Gemfile d'un lock, c'est son chemin sans .lock : captive-ruby/Gemfile.lock → captive-ruby/Gemfile.
+    def commande(lock) = "bundle lock --gemfile #{Shell.escape(lock.delete_suffix('.lock'))}"
   end
 end

data/lib/captive_release/git_commands.rb

@@ -4,8 +4,7 @@ module CaptiveRelease
   # Les commandes git d'une release. La façon de les lancer vit dans Shell.
   class GitCommands
     def self.dirty?
-      sortie = Shell.capture("git diff --name-only && git diff --cached --name-only")
-      !sortie.lines.map(&:strip).reject(&:empty?).empty?
+      !Shell.capture("git diff --name-only && git diff --cached --name-only").empty?
     end
 
     def self.remote_url = Shell.capture("git remote get-url origin 2>/dev/null")
@@ -19,7 +18,7 @@ module CaptiveRelease
     # Les sujets de commit depuis le tag donné jusqu'à HEAD - la matière première de la
     # suggestion de bump, un message par ligne, propre de toute ligne vide.
     def self.messages_since(tag)
-      Shell.capture("git log --pretty=%s #{Shell.escape(tag)}..HEAD").lines.map(&:strip).reject(&:empty?)
+      Shell.capture_lines("git log --pretty=%s #{Shell.escape(tag)}..HEAD")
     end
 
     def self.current_branch

data/lib/captive_release/release_command.rb

@@ -30,13 +30,15 @@ module CaptiveRelease
 
     def chemins_a_commiter
       chemins = @config.version_files.map(&:path)
-      return chemins unless (refresh = GemfileLockRefresh.new).needed?
+      refresh = GemfileLockRefresh.new
+      locks = refresh.lockfiles
+      return chemins if locks.empty?
 
       @terminal.say "\n#{@terminal.bold(Messages.t(:refreshing_lockfile))}"
-      refresh.call
-      @terminal.done GemfileLockRefresh::LOCKFILE
+      refresh.call(locks)
+      locks.each { |lock| @terminal.done lock }
 
-      chemins + [ GemfileLockRefresh::LOCKFILE ]
+      chemins + locks
     end
 
     def publication

data/lib/captive_release/shell.rb

@@ -13,6 +13,8 @@ module CaptiveRelease
 
     def self.capture(commande) = `#{commande}`.strip
 
+    def self.capture_lines(commande) = capture(commande).lines.map(&:strip).reject(&:empty?)
+
     # Rubocop veut Shellwords, mais son échappement diffère du nôtre sur des chemins que git
     # accepte : la liste blanche dit exactement ce qui passe sans protection.
     def self.escape(chaine) = chaine.gsub(%r{[^a-zA-Z0-9._\-/]}) { |caractere| "\\#{caractere}" }

data/lib/captive_release/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CaptiveRelease
-  VERSION = "0.8.0"
+  VERSION = "0.8.1"
 end

data/lib/captive_release/version_detector.rb

@@ -33,21 +33,22 @@ module CaptiveRelease
 
     def by_content(segments, type, pattern)
       @files.glob(*segments)
-            .select { |chemin| File.read(chemin).match?(pattern) }
+            .select { |chemin| contient?(chemin, pattern) }
             .map { |chemin| version_file(chemin, type) }
-    rescue StandardError
-      []
     end
 
     # Le build.gradle est à un chemin fixe : le chercher par glob remonterait ceux des
     # modules Android embarqués, qui ne portent pas la version de l'app.
     def android_gradle
-      chemin = @files.path(*GRADLE_PATH)
-      return [] unless File.exist?(chemin) && File.read(chemin).match?(/versionName\s+['"]/)
+      return [] unless contient?(@files.path(*GRADLE_PATH), /versionName\s+['"]/)
 
       [ VersionFile.new(GRADLE_PATH.join("/"), "android_gradle") ]
+    end
+
+    def contient?(chemin, pattern)
+      File.exist?(chemin) && File.read(chemin).match?(pattern)
     rescue StandardError
-      []
+      false
     end
 
     def version_file(chemin, type) = VersionFile.new(@files.relative(chemin), type)

data/lib/captive_release/version_file_selection.rb

@@ -13,7 +13,7 @@ module CaptiveRelease
 
     def call(found)
       list(found)
-      found.size == 1 ? seul(found) : exclude(found, demander)
+      found.size == 1 ? seul(found) : ExclusionInput.apply(found, demander)
     end
 
     private
@@ -37,13 +37,6 @@ module CaptiveRelease
       found
     end
 
-    def exclude(found, saisie)
-      return found if saisie.strip.empty?
-
-      rangs = saisie.split(",").map { |numero| numero.strip.to_i - 1 }
-      found.reject.with_index { |_, rang| rangs.include?(rang) }
-    end
-
     def readable_version(fichier)
       fichier.read_version
     rescue StandardError

data/lib/captive_release.rb

@@ -38,6 +38,7 @@ require_relative "captive_release/build_command"
 require_relative "captive_release/menu"
 require_relative "captive_release/version_choice"
 require_relative "captive_release/gemfile_lock_refresh"
+require_relative "captive_release/exclusion_input"
 require_relative "captive_release/git_release"
 require_relative "captive_release/changes_since"
 require_relative "captive_release/release_publication"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: captive-release
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Captive Studio
@@ -33,6 +33,7 @@ files:
 - lib/captive_release/config_file.rb
 - lib/captive_release/config_loader.rb
 - lib/captive_release/config_writer.rb
+- lib/captive_release/exclusion_input.rb
 - lib/captive_release/framework_detector.rb
 - lib/captive_release/gemfile_lock_refresh.rb
 - lib/captive_release/git_commands.rb