optparse2 0.7.3 → 0.7.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 231baf4fb3c4a278ea945a8bcff14e90decf66d13b37c25659cfc364ae9c0599
-  data.tar.gz: ede27c9be0e8bd48f9efe7358d716015d555a4f26544823354da0f005cb871ae
+  metadata.gz: 1fb6261613557a808dfda4002a86d4218d955804583135a3489ce5de01ba0e0e
+  data.tar.gz: babfce64dd51938c50999db68237a920a9270701c85b415e5fde510796134189
 SHA512:
-  metadata.gz: 7365315d5bf866353e233f5771057eb709871ad75d320ac979b860d5d6625cdcbb315cdc073eb6a79a0542c33025e320f77bbacde1101f9c641cf4b41dbafb30
-  data.tar.gz: 619232fed57b4670346be829b7d4afac06748f117b415eeaabe71be96f57a5870b0d32a8b3b5d222b70b8c432abfeed6b5acf6d701d38c0ca00dc395c221200e
+  metadata.gz: b7b7521fc75776437521cbf73fc024fc15ffa2f1a5b59ed7c43319f4fa8b103083e7e8eced9ae63d74475d741d49f84b43f4413cd806ceb41199223bd545ee0e
+  data.tar.gz: ef7eb6f76bd62d96d293303aed61d3ee884212145c6507cb373877b6fa19005c6bc5e6be6ce2c328ed963c11fee6b931c4dbbcdb68fbb2ef29d9fa04ff33d631

data/lib/optparse2/switch-helpers.rb

@@ -136,10 +136,25 @@ class OptParse2
       self.switch_name = val
     end
 
+    # Required arguments must be supplied by the user. They conflict with `default` options, and
+    # cause an error if both required and default are specified
+    def set_required(required)
+      if default?
+        raise ArgumentError, 'cannot require an argument with a default value'
+      end
+      @required = required
+    end
+
+    def required? = @required
+
     # Default values of switches are used when the switch is never passed in.
     # If the `value` that's provided doesn't respond to `.call`, it's converted to a proc.
     # If `bypass` is truthy, then the default value is never passed to the block's proc (if any)
     def set_default(value, description, bypass)
+      if required?
+        raise ArgumentError, 'cannot supply a default value for a required argument'
+      end
+
       if @arg.nil? && value != true && !bypass
         raise ArgumentError, "Cannot supply a non-true default value to a flag which takes no arguments", caller(4)
       end

data/lib/optparse2/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class OptParse2
-  VERSION = "0.7.3"
+  VERSION = "0.7.4"
 end

data/lib/optparse2.rb

@@ -18,7 +18,6 @@ class OptParse2
   def initialize(...)
     @defaults = Set[]
     @positional = []
-    @required = Set[]
     @rest = nil
     @group = nil
     self.pos_set_banner = OptParse2.pos_set_banner
@@ -55,12 +54,7 @@ class OptParse2
       not_style.set_hidden if hidden
     end
 
-    if required
-      unless nodefault
-        raise ArgumentError, "cannot supply both a default with required: true"
-      end
-      @required << sw.switch_name
-    end
+    sw.set_required true if required
 
     if nodefault && default_description != nil
       raise ArgumentError, "default: not supplied, but default_description: given"
@@ -147,8 +141,10 @@ class OptParse2
   end
 
   private def ensure_all_required_arguments_were_supplied!(context)
-    @required.each do |key|
-      raise ParseError, "required option '#{key}' not provided" unless context.key? key.to_sym
+    visit :each_option do |sw|
+      next unless sw.required?
+      key = sw.switch_name.to_sym
+      raise ParseError, "required option '#{key}' not provided" unless context.key? key
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: optparse2
 version: !ruby/object:Gem::Version
-  version: 0.7.3
+  version: 0.7.4
 platform: ruby
 authors:
 - SamW