rake-commander 0.2.5 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 580b47f2dc5213236846d91e47832787c2f1d50cd396dae322ea1ab91b6c31f7
-  data.tar.gz: 348609827f993bea3b61a9ca43d59456a98f093bd1029e1988bf2f2862633ee2
+  metadata.gz: ba9099907341e84e8420c7df7eca7f1b8f5279e454f73e031f01f982285ff583
+  data.tar.gz: 5263523e7800afd7a1a172eeafb5ae65e48a5b535c0ec6467b52656b90778c58
 SHA512:
-  metadata.gz: aae7e0db0dcd102dc43b4a1a5f1281be80ddb6246edaeb5507b1bf17d88f6cca7fcac31f090cf10a95176eaf880a891b937b5136fdcc374003ab0afc2daf622e
-  data.tar.gz: '0080e78b63488f7ce927e8364a479f3bc11d5f920708b0b640fd0fc3e7668d69edb3940df958a4cd3e667a0842d5c63ac6c89cf548b2f4e239b41db26e30d2fd'
+  metadata.gz: a5bdc0bcd00a882f0526e354f0a9a39b6efb0864748ffc088409ba7c2759f01ea2001d4f6eced4f56aaad2976d2ae5874a059e4e1cf434488243242e426444dd
+  data.tar.gz: 4fe1f53c6885a8b35b2193d96313c0f2e570cff131285def889a425110b8743a3fb792b2e7ae5646514870bfe15055cf529a0e49020219e5032f97a3f1315f96

data/CHANGELOG.md

@@ -32,11 +32,17 @@ All notable changes to this project will be documented in this file.
   - Option to globally enable/disable the 2nd patch?
     * That would make this gem completely useless.
 
-## [0.2.6] - 2023-05-xx
+## [0.2.7] - 2023-05-xx
 
 ### Added
 ### Fixed
 ### Changed
+  - `RakeCommander::Option` configure_other should only apply when the instance vars are not defined.
+
+## [0.2.6] - 2023-05-01
+
+### Changed
+  - `RakeCommander::Option` configure_other should only apply when the instance vars are not defined.
 
 ## [0.2.5] - 2023-05-01
 

data/lib/rake-commander/option.rb

@@ -125,11 +125,12 @@ class RakeCommander
 
     # @return [Hash] keyed arguments to create a new object
     def dup_key_arguments
+      configure_other
       {}.tap do |kargs|
         kargs.merge!(short:    short.dup.freeze)      if short
         kargs.merge!(name:     name_full.dup.freeze)  if name_full
-        kargs.merge!(desc:     desc.dup)              if desc
-        kargs.merge!(default:  default.dup)           if default?
+        kargs.merge!(desc:     @desc.dup)             if @desc
+        kargs.merge!(default:  @default.dup)          if default?
         kargs.merge!(type:     @type_coercion)        if @type_coercion.is_a?(Class)
         kargs.merge!(required: required?)
       end
@@ -228,17 +229,23 @@ class RakeCommander
 
     # It consumes `other_args`, to prevent direct overrides to be overriden by it.
     def configure_other
-      if type = other_args.find {|arg| arg.is_a?(Class)}
-        @type_coercion = type
-        other_args.delete(type)
-      end
-      if value = other_args.find {|arg| arg.is_a?(String)}
-        @desc = value
-        other_args.dup.each do |val|
-          other_args.delete(val) if val.is_a?(String)
-        end
-      end
+      @type_coertion ||= fetch_type_from_other
+      @desc          ||= fetch_desc_from_other
       nil
     end
+
+    def fetch_type_from_other
+      return nil unless type = other_args.find {|arg| arg.is_a?(Class)}
+      @type_coercion ||= type
+      other_args.delete(type)
+    end
+
+    def fetch_desc_from_other
+      return nil unless value = other_args.find {|arg| arg.is_a?(String)}
+      @desc ||= value
+      other_args.dup.each do |val|
+        other_args.delete(val) if val.is_a?(String)
+      end
+    end
   end
 end

data/lib/rake-commander/version.rb

@@ -1,3 +1,3 @@
 class RakeCommander
-  VERSION = '0.2.5'.freeze
+  VERSION = '0.2.6'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rake-commander
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.6
 platform: ruby
 authors:
 - Oscar Segura Samper