lino 1.8.0.pre.2 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d080aeff7401e2706a8d09877bb45f3b4f607289f9a422fe6e7cb8ae56f64a0c
-  data.tar.gz: 9b2b787947844e9af983a1e6f5eda10beb5dc9e13068dde2eb889dca429c89aa
+  metadata.gz: e889c057a237dfb1af77b0adbb0f661ef0cf37193341a6bddd8add27a64f12eb
+  data.tar.gz: 121d2e75f308626b29b1ac306f959583bd6cc9b18df0fb57b15d993f90f08dd5
 SHA512:
-  metadata.gz: '04596746b3bcee1f1d18ace01c68953cdac43070492f28bb262aee74b2ad21e8a8fdeb0ebb628ff7ebe1d61ac66ed4afa1e7aeb830556bd4175ac6206fd4f88b'
-  data.tar.gz: 496bf9bf680bb92c192670fb3a251fc2adeb80194baa558e3b4d76da6b9b366fa363111c0bfbe9137f851d638a85728f6b2f7bdfed05c5b27a926ee883b0e581
+  metadata.gz: 9f1319025180174dbce34687d0cbad930551e2ffad455da8b86ac31f47ba874aef4d012d3f4db23e1dd9de07f15e44a732bf4b937cd95cc1f559b18564fcea4d
+  data.tar.gz: 8a57fca71f15442bbef99be911bc6284f4e2f87c22425bdf996c09fb8c30429116c2bf3b27d6dca8f91e679f7240b9d1251048273f92c04ddb80c41de2c04b8e

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify dependencies in lino.gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    lino (1.8.0.pre.2)
+    lino (2.1.0)
       hamster (~> 3.0)
       open4 (~> 1.3)
 
@@ -16,6 +16,7 @@ GEM
       zeitwerk (~> 2.3)
     addressable (2.7.0)
       public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.2)
     colored2 (3.1.2)
     concurrent-ruby (1.1.7)
     diff-lcs (1.4.4)
@@ -37,7 +38,11 @@ GEM
       faraday (>= 0.9)
       sawyer (~> 0.8.0, >= 0.5.3)
     open4 (1.3.4)
+    parallel (1.20.1)
+    parser (3.0.0.0)
+      ast (~> 2.4.1)
     public_suffix (4.0.6)
+    rainbow (3.0.0)
     rake (13.0.3)
     rake_circle_ci (0.9.0)
       colored2 (~> 3.1)
@@ -59,6 +64,8 @@ GEM
       colored2 (~> 3.1)
       rake_factory (~> 0.23)
       sshkey (~> 2.0)
+    regexp_parser (2.1.1)
+    rexml (3.2.4)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)
@@ -72,6 +79,23 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
     rspec-support (3.10.1)
+    rubocop (1.12.0)
+      parallel (~> 1.10)
+      parser (>= 3.0.0.0)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.2.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.4.1)
+      parser (>= 2.7.1.5)
+    rubocop-rake (0.5.1)
+      rubocop
+    rubocop-rspec (2.2.0)
+      rubocop (~> 1.0)
+      rubocop-ast (>= 1.1.0)
+    ruby-progressbar (1.11.0)
     ruby2_keywords (0.0.2)
     ruby_gpg2 (0.6.0)
       lino (>= 1.5)
@@ -87,6 +111,7 @@ GEM
     sshkey (2.0.0)
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
+    unicode-display_width (2.0.0)
     zeitwerk (2.4.2)
 
 PLATFORMS
@@ -102,6 +127,9 @@ DEPENDENCIES
   rake_gpg (~> 0.12)
   rake_ssh (~> 0.4)
   rspec (~> 3.9)
+  rubocop (~> 1.12)
+  rubocop-rake (~> 0.5)
+  rubocop-rspec (~> 2.2)
   simplecov (~> 0.16)
 
 BUNDLED WITH

data/README.md

@@ -120,6 +120,18 @@ Lino::CommandLineBuilder.for_command('gcloud')
     .to_s
     
 # => gcloud sql instances set-root-password some-database --password super-secure
+
+# ... or alternatively
+Lino::CommandLineBuilder.for_command('gcloud')
+    .with_subcommands(
+      %w[sql instances set-root-password some-database]
+    ) do |sub|
+      sub.with_option('--password', 'super-secure')
+    end
+    .build
+    .to_s
+    
+# => gcloud sql instances set-root-password some-database --password super-secure
   
 # commands controlled by environment variables
 Lino::CommandLineBuilder.for_command('node')
@@ -183,10 +195,25 @@ To install dependencies and run the build, run the pre-commit build:
 ./go
 ```
 
-Tou can also run only the specs:
+This runs all unit tests and other checks including coverage and code linting / 
+formatting.
+
+To run only the unit tests, including coverage:
+
+```shell script
+./go test:unit
+```
+
+To attempt to fix any code linting / formatting issues:
+
+```shell script
+./go library:fix
+```
+
+To check for code linting / formatting issues without fixing:
 
 ```shell script
-./go spec
+./go library:check
 ```
 
 You can also run `bin/console` for an interactive prompt that will allow you to 

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'yaml'
 require 'rake_circle_ci'
 require 'rake_github'
@@ -5,10 +7,12 @@ require 'rake_ssh'
 require 'rake_gpg'
 require 'securerandom'
 require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
-task :default => :spec
-
-RSpec::Core::RakeTask.new(:spec)
+task default: %i[
+  library:fix
+  test:unit
+]
 
 namespace :encryption do
   namespace :passphrase do
@@ -23,83 +27,99 @@ end
 namespace :keys do
   namespace :deploy do
     RakeSSH.define_key_tasks(
-        path: 'config/secrets/ci/',
-        comment: 'maintainers@infrablocks.io')
+      path: 'config/secrets/ci/',
+      comment: 'maintainers@infrablocks.io'
+    )
   end
 
   namespace :gpg do
     RakeGPG.define_generate_key_task(
-        output_directory: 'config/secrets/ci',
-        name_prefix: 'gpg',
-        owner_name: 'InfraBlocks Maintainers',
-        owner_email: 'maintainers@infrablocks.io',
-        owner_comment: 'lino CI Key')
+      output_directory: 'config/secrets/ci',
+      name_prefix: 'gpg',
+      owner_name: 'InfraBlocks Maintainers',
+      owner_email: 'maintainers@infrablocks.io',
+      owner_comment: 'lino CI Key'
+    )
   end
 end
 
+RuboCop::RakeTask.new
+
+namespace :library do
+  desc 'Run all checks of the library'
+  task check: [:rubocop]
+
+  desc 'Attempt to automatically fix issues with the library'
+  task fix: [:'rubocop:auto_correct']
+end
+
+namespace :test do
+  RSpec::Core::RakeTask.new(:unit)
+end
+
 RakeCircleCI.define_project_tasks(
-    namespace: :circle_ci,
-    project_slug: 'github/infrablocks/lino'
+  namespace: :circle_ci,
+  project_slug: 'github/infrablocks/lino'
 ) do |t|
   circle_ci_config =
-      YAML.load_file('config/secrets/circle_ci/config.yaml')
+    YAML.load_file('config/secrets/circle_ci/config.yaml')
 
-  t.api_token = circle_ci_config["circle_ci_api_token"]
+  t.api_token = circle_ci_config['circle_ci_api_token']
   t.environment_variables = {
-      ENCRYPTION_PASSPHRASE:
-          File.read('config/secrets/ci/encryption.passphrase')
-              .chomp
+    ENCRYPTION_PASSPHRASE:
+        File.read('config/secrets/ci/encryption.passphrase')
+            .chomp
   }
   t.checkout_keys = []
   t.ssh_keys = [
-      {
-          hostname: "github.com",
-          private_key: File.read('config/secrets/ci/ssh.private')
-      }
+    {
+      hostname: 'github.com',
+      private_key: File.read('config/secrets/ci/ssh.private')
+    }
   ]
 end
 
 RakeGithub.define_repository_tasks(
-    namespace: :github,
-    repository: 'infrablocks/lino',
+  namespace: :github,
+  repository: 'infrablocks/lino'
 ) do |t|
   github_config =
-      YAML.load_file('config/secrets/github/config.yaml')
+    YAML.load_file('config/secrets/github/config.yaml')
 
-  t.access_token = github_config["github_personal_access_token"]
+  t.access_token = github_config['github_personal_access_token']
   t.deploy_keys = [
-      {
-          title: 'CircleCI',
-          public_key: File.read('config/secrets/ci/ssh.public')
-      }
+    {
+      title: 'CircleCI',
+      public_key: File.read('config/secrets/ci/ssh.public')
+    }
   ]
 end
 
 namespace :pipeline do
-  task :prepare => [
-      :'circle_ci:project:follow',
-      :'circle_ci:env_vars:ensure',
-      :'circle_ci:checkout_keys:ensure',
-      :'circle_ci:ssh_keys:ensure',
-      :'github:deploy_keys:ensure'
+  task prepare: %i[
+    circle_ci:project:follow
+    circle_ci:env_vars:ensure
+    circle_ci:checkout_keys:ensure
+    circle_ci:ssh_keys:ensure
+    github:deploy_keys:ensure
   ]
 end
 
 namespace :version do
-  desc "Bump version for specified type (pre, major, minor, patch)"
+  desc 'Bump version for specified type (pre, major, minor, patch)'
   task :bump, [:type] do |_, args|
     bump_version_for(args.type)
   end
 end
 
-desc "Release gem"
+desc 'Release gem'
 task :release do
-  sh "gem release --tag --push"
+  sh 'gem release --tag --push'
 end
 
 def bump_version_for(version_type)
-  sh "gem bump --version #{version_type} " +
-      "&& bundle install " +
-      "&& export LAST_MESSAGE=\"$(git log -1 --pretty=%B)\" " +
-      "&& git commit -a --amend -m \"${LAST_MESSAGE} [ci skip]\""
+  sh "gem bump --version #{version_type} " \
+     '&& bundle install ' \
+     '&& export LAST_MESSAGE="$(git log -1 --pretty=%B)" ' \
+     '&& git commit -a --amend -m "${LAST_MESSAGE} [ci skip]"'
 end

data/bin/console

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require 'bundler/setup'
 require 'lino'

data/lib/lino.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'lino/version'
 require 'lino/command_line'
 require 'lino/command_line_builder'

data/lib/lino/command_line.rb

@@ -1,24 +1,28 @@
+# frozen_string_literal: true
+
 require 'open4'
 
 module Lino
   class CommandLine
-    def initialize command_line
+    def initialize(command_line)
       @command_line = command_line
     end
 
     def execute(
-        stdin: '',
-        stdout: STDOUT,
-        stderr: STDERR)
-      Open4::spawn(
-          @command_line,
-          stdin: stdin,
-          stdout: stdout,
-          stderr: stderr)
+      stdin: '',
+      stdout: $stdout,
+      stderr: $stderr
+    )
+      Open4.spawn(
+        @command_line,
+        stdin: stdin,
+        stdout: stdout,
+        stderr: stderr
+      )
     end
 
     def to_s
       @command_line
     end
   end
-end
+end

data/lib/lino/command_line_builder.rb

@@ -1,24 +1,28 @@
+# frozen_string_literal: true
+
 require 'hamster'
 require_relative 'utilities'
 require_relative 'command_line'
 require_relative 'subcommand_builder'
-require_relative 'switches'
+require_relative 'options'
 
 module Lino
+  # rubocop:disable Metrics/ClassLength
   class CommandLineBuilder
     include Lino::Utilities
-    include Lino::Switches
+    include Lino::Options
 
-    class <<self
+    class << self
       def for_command(command)
         CommandLineBuilder.new(command: command)
       end
     end
 
+    # rubocop:disable Metrics/ParameterLists
     def initialize(
       command: nil,
       subcommands: [],
-      switches: [],
+      options: [],
       arguments: [],
       environment_variables: [],
       option_separator: ' ',
@@ -26,12 +30,13 @@ module Lino
     )
       @command = command
       @subcommands = Hamster::Vector.new(subcommands)
-      @switches = Hamster::Vector.new(switches)
+      @options = Hamster::Vector.new(options)
       @arguments = Hamster::Vector.new(arguments)
       @environment_variables = Hamster::Vector.new(environment_variables)
       @option_separator = option_separator
       @option_quoting = option_quoting
     end
+    # rubocop:enable Metrics/ParameterLists
 
     def with_subcommand(subcommand, &block)
       with(
@@ -43,6 +48,15 @@ module Lino
       )
     end
 
+    def with_subcommands(subcommands, &block)
+      without_block = subcommands[0...-1]
+      with_block = subcommands.last
+
+      without_block
+        .inject(self) { |s, sc| s.with_subcommand(sc) }
+        .with_subcommand(with_block, &block)
+    end
+
     def with_option_separator(option_separator)
       with(option_separator: option_separator)
     end
@@ -52,32 +66,33 @@ module Lino
     end
 
     def with_argument(argument)
-      with(arguments: add_argument(argument))
+      return self if missing?(argument)
+
+      with(arguments: @arguments.add({ components: [argument] }))
     end
 
     def with_arguments(arguments)
-      arguments.each { |argument| add_argument(argument) }
-      with({})
+      arguments.inject(self) { |s, argument| s.with_argument(argument) }
     end
 
     def with_environment_variable(environment_variable, value)
-      with(environment_variables: @environment_variables.add([environment_variable, value]))
+      with(
+        environment_variables:
+          @environment_variables.add(
+            [
+              environment_variable, value
+            ]
+          )
+      )
     end
 
     def build
       components = [
-        map_and_join(@environment_variables) do |var|
-          "#{var[0]}=\"#{var[1].to_s.gsub(/"/, '\\"')}\""
-        end,
+        formatted_environment_variables,
         @command,
-        map_and_join(
-          @switches,
-          &(quote_with(@option_quoting) >> join_with(@option_separator))
-        ),
-        map_and_join(@subcommands) do |sub|
-          sub.build(@option_separator, @option_quoting)
-        end,
-        map_and_join(@arguments, &join_with(' '))
+        formatted_options,
+        formatted_subcommands,
+        formatted_arguments
       ]
 
       command_string = components.reject(&:empty?).join(' ')
@@ -87,6 +102,32 @@ module Lino
 
     private
 
+    def formatted_environment_variables
+      map_and_join(@environment_variables) do |var|
+        "#{var[0]}=\"#{var[1].to_s.gsub(/"/, '\\"')}\""
+      end
+    end
+
+    def formatted_options
+      map_and_join(
+        @options,
+        &(quote_with(@option_quoting) >> join_with(@option_separator))
+      )
+    end
+
+    def formatted_subcommands
+      map_and_join(@subcommands) do |sub|
+        sub.build(@option_separator, @option_quoting)
+      end
+    end
+
+    def formatted_arguments
+      map_and_join(
+        @arguments,
+        &join_with(' ')
+      )
+    end
+
     def with(**replacements)
       CommandLineBuilder.new(**state.merge(replacements))
     end
@@ -95,18 +136,13 @@ module Lino
       {
         command: @command,
         subcommands: @subcommands,
-        switches: @switches,
+        options: @options,
         arguments: @arguments,
         environment_variables: @environment_variables,
         option_separator: @option_separator,
         option_quoting: @option_quoting
       }
     end
-
-    def add_argument(argument)
-      return @arguments if missing?(argument)
-
-      @arguments = @arguments.add({ components: [argument] })
-    end
   end
+  # rubocop:enable Metrics/ClassLength
 end

data/lib/lino/options.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module Lino
+  module Options
+    def with_option(option, value, separator: nil, quoting: nil)
+      return self if missing?(value)
+
+      with(options: @options.add(
+        {
+          components: [option, value],
+          separator: separator,
+          quoting: quoting
+        }
+      ))
+    end
+
+    def with_repeated_option(option, values, separator: nil, quoting: nil)
+      values.inject(self) do |s, value|
+        s.with_option(option, value, separator: separator, quoting: quoting)
+      end
+    end
+
+    def with_flag(flag)
+      with(options: @options.add({ components: [flag] }))
+    end
+  end
+end

data/lib/lino/subcommand_builder.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 require 'hamster'
 require_relative 'utilities'
-require_relative 'switches'
+require_relative 'options'
 
 module Lino
   class SubcommandBuilder
     include Lino::Utilities
-    include Lino::Switches
+    include Lino::Options
 
     class <<self
       def for_subcommand(subcommand)
@@ -13,16 +15,16 @@ module Lino
       end
     end
 
-    def initialize(subcommand: nil, switches: [])
+    def initialize(subcommand: nil, options: [])
       @subcommand = subcommand
-      @switches = Hamster::Vector.new(switches)
+      @options = Hamster::Vector.new(options)
     end
 
     def build(option_separator, option_quoting)
       components = [
         @subcommand,
         map_and_join(
-          @switches,
+          @options,
           &(quote_with(option_quoting) >> join_with(option_separator))
         )
       ]
@@ -38,7 +40,7 @@ module Lino
     def state
       {
         subcommand: @subcommand,
-        switches: @switches
+        options: @options
       }
     end
   end

data/lib/lino/utilities.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pp'
 
 module Lino
@@ -14,20 +16,30 @@ module Lino
 
     def quote_with(global_character)
       lambda do |item|
-        character = item[:quoting] || global_character
-        components = item[:components]
-        switch = components[0]
-        value = components[1]
-
         item.merge(
-            components: (components.count > 1) ?
-                [switch, "#{character}#{value}#{character}"] :
-                components)
+          components: resolve_components(item, global_character)
+        )
       end
     end
 
     def missing?(value)
       value.nil? || (value.respond_to?(:empty?) && value.empty?)
     end
+
+    private
+
+    def resolve_components(item, global_character)
+      components = item[:components]
+      switch = components[0]
+
+      if components.count > 1
+        character = item[:quoting] || global_character
+        value = components[1]
+
+        [switch, "#{character}#{value}#{character}"]
+      else
+        components
+      end
+    end
   end
-end
+end

data/lib/lino/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Lino
-  VERSION = '1.8.0.pre.2'
+  VERSION = '2.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lino
 version: !ruby/object:Gem::Version
-  version: 1.8.0.pre.2
+  version: 2.1.0
 platform: ruby
 authors:
 - Toby Clemson
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-03-30 00:00:00.000000000 Z
+date: 2021-04-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hamster
@@ -52,6 +52,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: gem-release
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -95,33 +109,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.5'
 - !ruby/object:Gem::Dependency
-  name: rake_ssh
+  name: rake_gpg
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.4'
+        version: '0.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.4'
+        version: '0.12'
 - !ruby/object:Gem::Dependency
-  name: rake_gpg
+  name: rake_ssh
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.12'
+        version: '0.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.12'
+        version: '0.4'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -137,19 +151,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.9'
 - !ruby/object:Gem::Dependency
-  name: gem-release
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.12'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.2'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -182,8 +224,8 @@ files:
 - lib/lino.rb
 - lib/lino/command_line.rb
 - lib/lino/command_line_builder.rb
+- lib/lino/options.rb
 - lib/lino/subcommand_builder.rb
-- lib/lino/switches.rb
 - lib/lino/utilities.rb
 - lib/lino/version.rb
 homepage: https://github.com/infrablocks/lino
@@ -201,9 +243,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.0.1
 signing_key: 

data/lib/lino/switches.rb

@@ -1,32 +0,0 @@
-module Lino
-  module Switches
-    def with_option(switch, value, separator: nil, quoting: nil)
-      with(switches: add_option(switch, value, separator, quoting))
-    end
-
-    def with_repeated_option(switch, values, separator: nil, quoting: nil)
-      values.each do |value|
-        add_option(switch, value, separator, quoting)
-      end
-      with({})
-    end
-
-    def with_flag(flag)
-      with(switches: @switches.add({ components: [flag] }))
-    end
-
-    private
-
-    def add_option(switch, value, separator, quoting)
-      return @switches if missing?(value)
-
-      @switches = @switches.add(
-        {
-          components: [switch, value],
-          separator: separator,
-          quoting: quoting
-        }
-      )
-    end
-  end
-end