Currently, there is no way to implement parsing of commands that accept arguments. Let

How about something like this: <div class="highlight highlight-source-ruby notrans

Closed by <a class="issue-link js-issue-link" data-error-text="Failed to load title" d

Commands should support arguments about slop HOT 7 CLOSED

leejarvis commented on May 19, 2024

Commands should support arguments

from slop.

Comments (7)

leejarvis commented on May 19, 2024

Thanks for reporting this. I should have some time tomorrow to look into it a little more.

from slop.

leejarvis commented on May 19, 2024

How about something like this:

require 'slop'

opts = Slop::Commands.new do
  on :install
end

opts.parse %w(install geronimo)

p opts.arguments #=> ["geronimo"]

from slop.

kyrylo commented on May 19, 2024

Looks good, but still there is no way to tell, which command has been triggered:

opts = Slop::Commands.new do
  on :install
  on :list
end

opts.parse %w(install geronimo)

p opts.arguments #=> ["geronimo"]
# Is this an argument of `install` or maybe `list`?

from slop.

leejarvis commented on May 19, 2024

I can't see why you would ever care which command was used. I see something like this:

if opts[:install]
  run_installer(opts.arguments)
end

And just pass around opts.arguments to any sub command you need

from slop.

kyrylo commented on May 19, 2024

I don't know, maybe I don't understand something, but this if check doesn't work as you showed in your example. opts[:install] always returns a Slop instance, whatever happens.

opts = Slop::Commands.new do
  on :install
  on :list
end

opts.parse %w(install geronimo)

p opts[:install]
#=> #<Slop {:strict=>false, :help=>false, :banner=>nil, :ignore_case=>false, :autocreate=>false, :arguments=>false, :optional_arguments=>false, :multiple_switches=>true, :longest_flag=>0} []>

p opts[:list]
#=> #<Slop {:strict=>false, :help=>false, :banner=>nil, :ignore_case=>false, :autocreate=>false, :arguments=>false, :optional_arguments=>false, :multiple_switches=>true, :longest_flag=>0} []>

P.S.: Also, keep in mind the situation when we want to pass multiple arguments (like we can do it with options: --install geronimo,banzai,huzzah)

from slop.

leejarvis commented on May 19, 2024

Yeah you're right. I think temporarily something like this will have to do:

opts = Slop::Commands.new do
  on :install
  on :list
end

opts.parse %w(install geronimo)

if opts.present?(:install)
  # do something with opts.arguments
end