rake-commander 0.2.4 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e676c7895883ca91c58c1ef5a025a4a8f52795ed12ac79e2e53f304b59e5a9ee
-  data.tar.gz: 871c881ac31b06e1cf4a13bab7846d063bfee83bfcca4b332eeeb345f04c3051
+  metadata.gz: ba9099907341e84e8420c7df7eca7f1b8f5279e454f73e031f01f982285ff583
+  data.tar.gz: 5263523e7800afd7a1a172eeafb5ae65e48a5b535c0ec6467b52656b90778c58
 SHA512:
-  metadata.gz: 78d4f8199e7e220d26e48346fd0cd23aec273c739ee8bcd949ed12cf147191ec988f056d6ef273f5dc186e64bb71a2bc7eb33dbcbc2525f2d20278dac3ce1df2
-  data.tar.gz: 31a2dca9846b7521b43ca14d79a562f57a30fafad01af42e3a0886e4aceb3b4d43d71f8b6133f08ca538cbf9e6d5210ed82447dbfb20f70ea8ab315797dedae7
+  metadata.gz: a5bdc0bcd00a882f0526e354f0a9a39b6efb0864748ffc088409ba7c2759f01ea2001d4f6eced4f56aaad2976d2ae5874a059e4e1cf434488243242e426444dd
+  data.tar.gz: 4fe1f53c6885a8b35b2193d96313c0f2e570cff131285def889a425110b8743a3fb792b2e7ae5646514870bfe15055cf529a0e49020219e5032f97a3f1315f96

data/CHANGELOG.md

@@ -32,11 +32,22 @@ 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.5] - 2023-04-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
+
+### Changed
+  - `RakeCommander::Base::ClassInheritable#inherited_class_value` now expects a reference to the subclass
 
 ## [0.2.4] - 2023-04-30
 

data/lib/rake-commander/base/class_inheritable.rb

@@ -50,7 +50,7 @@ class RakeCommander
             definitions.each do |var, action|
               instance_var = instance_variable_name(var)
               value        = instance_variable_get(instance_var)
-              child_value  = inherited_class_value(value, method, action)
+              child_value  = inherited_class_value(value, method, action, subclass)
               subclass.instance_variable_set(instance_var, child_value)
             end
           end
@@ -58,14 +58,14 @@ class RakeCommander
       end
 
       # @return [Variant] the value that the child class will inherit
-      def inherited_class_value(value, method, action)
+      def inherited_class_value(value, method, action, subclass)
         case method
         when :mirror
           value
         when :deep_dup
           case action
           when Proc
-            action.call(value)
+            action.call(value, subclass)
           when :default
             custom_deep_dup(value)
           end

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.4'.freeze
+  VERSION = '0.2.6'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rake-commander
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.6
 platform: ruby
 authors:
 - Oscar Segura Samper
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-04-29 00:00:00.000000000 Z
+date: 2023-05-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler