homebrew_automation 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9a3d541470864fdabac7601c9b4bca62c97f3b8a94f1af65d8dd36ab5bdd93d9
-  data.tar.gz: 23d0c6a2f2e625ace46a36bbdf18b13cd43611e66bf52655ea098208bf5d1dc0
+  metadata.gz: 7857fc3a55cf794b0138e422f94c7ca6d259ba9cca51df74c46a07c2986231bb
+  data.tar.gz: d2c5d3a431e0983dafbd8d3504bfea58b6e2c4673272260f18a69ffba8fe3d76
 SHA512:
-  metadata.gz: 175915af38029378ecdc8b78fd463eb55e14e34d6d18222479f2feca5883c8f6f46341a68bba2ab44125b16121ec38392e0caaae973c45b32e86622dc69dea76
-  data.tar.gz: 362510628786c45559de477492efb7857ada4b3bd6b6973733c662b5307f0e550a2f515c87f80b2cde57a2cbc65c13fe5dde16a02fdc844c225aabb16aeb0871
+  metadata.gz: 8a6ee4421b0b4a460b6cd6fc818b1e72952fdd1e0d17522cb1ec0787d28e66b05dbfae92bd21638c68a7a1a2c305a4fe52c4ff2dec52a0e35883836311879371
+  data.tar.gz: e0c4ae780bc69a17951f9d248a0b29e99a03f06f2f2343332f11d5693d343ab182fd18549a3f8cf3dcfc3d99efb2a080efba81fb25ab10e0b0243874ccc3bac3

data/lib/homebrew_automation/bintray/version.rb

@@ -11,6 +11,10 @@ module HomebrewAutomation::Bintray
   # As per Bintray, a +Version+ is part of a +Package+ is part of a +Repository+.
   class Version
 
+    # The file to upload already exists on Bintray
+    class FileAlreadyExists < StandardError
+    end
+
     # @param client [Client] Connection to Bintray servers
     # @param repo_name [String]
     # @param package_name [String]
@@ -39,12 +43,21 @@ module HomebrewAutomation::Bintray
     # @param filename [String]
     # @param content [String] the bytes in the file
     def upload_file!(filename, content)
-      @client.upload_file(
-        @repo_name,
-        @package_name,
-        @version_name,
-        filename,
-        content)
+      begin
+        @client.upload_file(
+          @repo_name,
+          @package_name,
+          @version_name,
+          filename,
+          content)
+      rescue RestClient::ExceptionWithResponse => e
+        case e.response.code
+        when 409
+          raise FileAlreadyExists
+        else
+          raise e
+        end
+      end
     end
 
     # Download metadata about files that exist on Bintray for this +Version+

data/lib/homebrew_automation/bottle.rb

@@ -1,103 +1,116 @@
 
 require 'json'
 
+require_relative './brew.rb'
+
 module HomebrewAutomation
 
-  # A representation of a binary build of a Homebrew package
+  # Metadata for building a Bottle for a Homebrew package
   class Bottle
 
+    class Error < StandardError
+    end
+
+    # @param tap_name [String] For use with +brew tap+
     # @param tap_url [String] Something suitable for +git clone+, e.g. +git@github.com:easoncxz/homebrew-tap.git+ or +/some/path/to/my-git-repo+
     # @param formula_name [String] As known by Homebrew
     # @param os_name [String] As known by Homebrew, e.g. +el_capitan+
-    # @param filename [String] ???
-    # @param content [String] ???
     # @param keep_tmp [Boolean] pass +--keep-tmp+ to +brew+
+    # @param brew [Brew] Homebrew effects
+    # @param bottle_finder [Bottle] Bottle-related filesystem effects
     def initialize(
+        tap_name,
         tap_url,
         formula_name,
         os_name,
-        filename: nil,
-        content: nil,
-        keep_tmp: false)
+        keep_tmp: false,
+        brew: Brew,
+        bottle_finder: Bottle)
+      @tap_name = tap_name
       @tap_url = tap_url
       @formula_name = formula_name
       @os_name = os_name
-      @filename = filename
-      @minus_minus = nil  # https://github.com/Homebrew/brew/pull/4612
-      @content = content
       @keep_tmp = keep_tmp
+      @brew = brew
+      @bottle_finder = bottle_finder
     end
 
-    # Takes ages to run, just like if done manually
-    #
-    # @raise [StandardError]
-    # @return [nil]
-    def build
-      complain unless system 'brew', 'tap', tmp_tap_name, @tap_url
-      maybe_keep_tmp = @keep_tmp ? ['--keep-tmp'] : []
-      install_cmd = ['brew', 'install', '--verbose'] + maybe_keep_tmp + ['--build-bottle', fully_qualified_formula_name]
-      complain unless system(*install_cmd)
-      complain unless system 'brew', 'bottle', '--verbose', '--json', '--no-rebuild', fully_qualified_formula_name
-    end
-
-    # Read and analyse metadata JSON file
-    # @return [Array<(String, String)>] {#minus_minus} and {#filename}
-    def locate_tarball
-      json_filename = Dir['*.bottle.json'].first
-      unless json_filename
-        build
-        return locate_tarball
-      end
-      json = JSON.parse(File.read(json_filename))
-      focus = json || complain
-      focus = focus[json.keys.first] || complain
-      focus = focus['bottle'] || complain
-      focus = focus['tags'] || complain
-      focus = focus[@os_name] || complain
-      @minus_minus, @filename = focus['local_filename'], focus['filename']
-    end
-
-    # The +brew bottle+ original output filename
+    # Build the bottle and get a binary tarball suitable for upload to Bintray
     #
-    # See https://github.com/Homebrew/brew/pull/4612 for details.
+    # Unless you've already run +brew install --build-bottle+ on that Formula
+    # on your system before, the returned effect would take ages to run (looking
+    # at about 30-60 minutes).
     #
-    # @return [String]
-    def minus_minus
-      @minus_minus || locate_tarball.first
-    end
-
-    # Filename of a Bottle tarball suitable for writing into a Formula file
-    #
-    # @return [String]
-    def filename
-      @filename || locate_tarball.last
+    # @yieldparam filename [String] A filename to tell Bintray which Homebrew can
+    #   recognise
+    # @yieldparam contents [String] The data of the binary Bottle tarball, as if
+    #   read via {File#read}
+    # @return [NilClass]
+    # @raise [Error]
+    def build!(&block)
+      raise Error, "Bottle#build! expects a block" unless block
+      call_brew! do
+        json_str = @bottle_finder.read_json!
+        (minus_minus, filename) = parse_for_tarball_path(json_str)
+        contents = @bottle_finder.read_tarball! minus_minus
+        block.call(filename, contents)
+      end
     end
 
-    # @return [String] {#content}
-    def load_tarball_from_disk
-      File.rename minus_minus, filename
-      @content = File.read filename
+    def self.read_json!
+      json_filename = Dir['*.bottle.json'].first
+      File.read(json_filename)
     end
 
-    # @return [String] bytes of the tarball of this Bottle
-    def content
-      @content || load_tarball_from_disk
+    def self.read_tarball!(minus_minus)
+      File.read(minus_minus)
     end
 
     private
 
-    # A name for the temporary tap; doesn't really matter what this is.
-    def tmp_tap_name
-      'easoncxz/tmp-tap'
+    # tap, install, and bottle
+    def call_brew!(&block)
+      tapped = false
+      begin
+        @brew.tap!(@tap_name, @tap_url)
+        tapped = true
+        @brew.install!(
+          %w[--verbose --build-bottle] + if @keep_tmp then %w[--keep-tmp] else [] end,
+          fully_qualified_formula_name)
+        @brew.bottle!(
+          %w[--verbose --json --no-rebuild],
+          fully_qualified_formula_name)
+        block.call
+      ensure
+        @brew.untap! @tap_name if tapped
+      end
     end
 
-    def fully_qualified_formula_name
-      tmp_tap_name + '/' + @formula_name
+    # pure-ish; raises exception
+    #
+    # @return [Tuple<String, String>] +[minus_minus, filename]+
+    def parse_for_tarball_path(json_str)
+      begin
+        focus = JSON.parse(json_str)
+        [fully_qualified_formula_name, 'bottle', 'tags', @os_name].each do |key|
+          focus = focus[key]
+          if focus.nil?
+            raise BottleError.new "unexpected JSON structure, couldn't find key: #{key}"
+          end
+        end
+        # https://github.com/Homebrew/brew/pull/4612
+        minus_minus, filename = focus['local_filename'], focus['filename']
+        if minus_minus.nil? || filename.nil?
+          raise BottleError.new "unexpected JSON structure, couldn't find both `local_filename` and `filename` keys: #{minus_minus.inspect}, #{filename.inspect}"
+        end
+        [minus_minus, filename]
+      rescue JSON::ParserError => e
+        raise BottleError.new "error parsing JSON: #{e}"
+      end
     end
 
-    def complain
-      puts "HEY! Something has gone wrong and I need to complain. Stacktrace follows:"
-      puts caller
+    def fully_qualified_formula_name
+      @tap_name + '/' + @formula_name
     end
 
   end

data/lib/homebrew_automation/brew.rb

@@ -0,0 +1,50 @@
+
+module HomebrewAutomation
+
+  # Homebrew effects
+  class Brew
+
+    class Error < StandardError
+    end
+
+    # +brew tap "$name" "$url"+
+    #
+    # @param name [String]
+    # @param url [String]
+    def self.tap!(name, url)
+      checked('brew', 'tap', name, url)
+    end
+
+    # +brew untap "$name"+
+    #
+    # @param name [String]
+    def self.untap!(name)
+      checked('brew', 'untap', name)
+    end
+
+    # +brew install [opts] "$fully_qualified_formula_name"+
+    #
+    # @param opts [Array<String>]
+    # @param fully_qualified_formula_name [String]
+    def self.install!(opts, fully_qualified_formula_name)
+      checked('brew', 'install', *opts, fully_qualified_formula_name)
+    end
+
+    # +brew bottle [opts] "$fully_qualified_formula_name"+
+    #
+    # @param opts [Array<String>]
+    # @param fully_qualified_formula_name [String]
+    def self.bottle!(opts, fully_qualified_formula_name)
+      checked('brew', 'bottle', *opts, fully_qualified_formula_name)
+    end
+
+    private_class_method def self.checked(*args)
+      result = system(*args)
+      unless result
+        raise BrewError.new("Command failed: #{args}")
+      end
+      result
+    end
+
+  end
+end

data/lib/homebrew_automation/cli/workflow_commands.rb

@@ -21,19 +21,21 @@ class HomebrewAutomation::CLI::WorkflowCommands < Thor
   class_option :bintray_repo
   class_option :bintray_package
   class_option :bintray_version
-  class_option :keep_tap_repo, :type => :boolean
 
   desc 'build-and-upload', 'Build binary tarball from source tarball, then upload to Bintray'
   long_desc <<-HERE_HERE
     Since we're uploading to Bintray, we need a Bintray API KEY at `bintray_token`.
   HERE_HERE
+  option :keep_tap_repo, :type => :boolean
   option :keep_brew_tmp, :type => :boolean
   def build_and_upload
-    workflow.build_and_upload_bottle(
+    workflow.build_and_upload_bottle!(
       sdist,
       tap,
+      git,
       formula_name,
       bintray_version,
+      keep_tap_repo: options[:keep_tap_repo],
       keep_homebrew_tmp: options[:keep_brew_tmp])
   end
 
@@ -45,7 +47,11 @@ class HomebrewAutomation::CLI::WorkflowCommands < Thor
     Tap repo on Github via a Github OAuth token via `tap_token`.
   HERE_HERE
   def gather_and_publish
-    workflow.gather_and_publish_bottles(sdist, tap, formula_name, bintray_version)
+    workflow.gather_and_publish_bottles!(
+      sdist,
+      tap,
+      formula_name,
+      bintray_version)
   end
 
   private
@@ -57,16 +63,15 @@ class HomebrewAutomation::CLI::WorkflowCommands < Thor
       options[:source_tag])
   end
 
-  def keep_tap_repo?
-    options[:keep_tap_repo]
-  end
-
   def tap
     HomebrewAutomation::Tap.new(
       options[:tap_user],
       options[:tap_repo],
-      options[:tap_token],
-      keep_submodule: keep_tap_repo?)
+      options[:tap_token])
+  end
+
+  def git
+    HomebrewAutomation::Git
   end
 
   # DOC: default values here

data/lib/homebrew_automation/git.rb

@@ -0,0 +1,116 @@
+
+require 'fileutils'
+
+module HomebrewAutomation
+
+  # Git effects
+  class Git
+
+    class Error < StandardError
+    end
+
+    class << self
+
+      # Set Git user's name and email
+      #
+      # Reads environment variables:
+      # - TRAVIS_GIT_USER_NAME
+      # - TRAVIS_GIT_USER_EMAIL
+      #
+      # If either env var is not set, do nothing.
+      #
+      # @return [NilClass]
+      def config!
+        name = ENV['TRAVIS_GIT_USER_NAME']
+        email = ENV['TRAVIS_GIT_USER_EMAIL']
+        if name && email
+          raise_unless 'git', 'config', '--global', 'user.name', name
+          raise_unless 'git', 'config', '--global', 'user.email', email
+        end
+      end
+
+      # Just +git clone+ the given URL
+      #
+      # @param url [String] git-friendly URL; could be filesystem path
+      # @param dir [String] optionally specify target dir name
+      # @return [NilClass]
+      def clone!(url, dir: nil)
+        if dir
+          raise_unless 'git', 'clone', url, dir
+        else
+          raise_unless 'git', 'clone', url
+        end
+      end
+
+      # Like {#clone!} , but allows you to do something inside
+      # the newly cloned directory, pushd-style.
+      #
+      # @see clone!
+      # @param url [String]
+      # @param dir [String]
+      # @param keep_dir [Boolean]
+      # @yieldparam dir [String] name of freshly cloned dir
+      # @yieldreturn [a] anything
+      # @return [a]
+      def with_clone!(url, dir, keep_dir: false, &block)
+        begin
+          clone! url, dir: dir
+          if block
+            Dir.chdir dir do
+              block.call(File.realpath '.')
+            end
+          else
+            puts "Strange, you're calling Git#with_clone! without a block."
+          end
+          nil
+        ensure
+          FileUtils.remove_dir(dir) unless keep_dir
+        end
+      end
+
+      # +git commit --allow-empty -am "$msg"+
+      #
+      # @param msg [String] Git commit message
+      # @return [NilClass]
+      def commit_am!(msg)
+        raise_unless 'git', 'commit', '--allow-empty', '-am', msg
+      end
+
+      # Just +git push+
+      #
+      # @return [NilClass]
+      def push!
+        raise_unless 'git', 'push'
+      end
+
+      private
+
+      def raise_unless(*args)
+        begin
+          result = system(*args)
+          unless result
+            raise Error, "Git command failed: #{args}"
+          end
+          result
+        end
+      end
+
+      # Impure
+      def complain_unless(*args)
+        result = system(*args)
+        unless result
+          puts "Git command errored: #{args}"
+          puts caller
+        end
+        result
+      end
+
+      def silently(*args)
+        system(*args)
+      end
+
+    end
+
+  end
+
+end

data/lib/homebrew_automation/mac_os.rb

@@ -9,11 +9,11 @@ module HomebrewAutomation
     # Return a macOS version name in a convention recognised by Homebrew, in
     # particular by the Formula/Bottle DSL.
     #
-    # @return [String]
-    def self.identify_version
+    # @return [String | NilClass]
+    def self.identify_version!
       version =
         begin
-          `sw_vers -productVersion`
+          `sw_vers -productVersion`.chomp
         rescue Errno::ENOENT    # if we're not on a Mac
           nil
         end

data/lib/homebrew_automation/tap.rb

@@ -6,27 +6,23 @@ module HomebrewAutomation
   # A representation of a Github repo that acts as a Homebrew Tap.
   class Tap
 
-    # Assign params to attributes.
-    #
     # See {#user}, {#repo}, {#token}.
-    #
-    # @param keep_submodule [Boolean] Avoid deleting the cloned tap Git repo
-    #   directory when possible
-    def initialize(user, repo, token, keep_submodule: false)
+    def initialize(user, repo, token)
+      @user = user
       @repo = repo
+      @token = token
       @url = "https://#{token}@github.com/#{user}/#{repo}.git"
-      @keep_submodule = keep_submodule
     end
 
-    # Github repo name, as appears in Github URLs
+    # Github username
     #
     # @return [String]
-    attr_reader :repo
+    attr_reader :user
 
-    # Repo URL, as expected by Git
+    # Github repo name, as appears in Github URLs
     #
     # @return [String]
-    attr_reader :url
+    attr_reader :repo
 
     # Github OAuth token
     #
@@ -35,37 +31,27 @@ module HomebrewAutomation
     # @return [String]
     attr_reader :token
 
-    # +pushd+ into a fresh clone, call the block, then clean-up.
-    #
-    # Haskell-y type: +forall a. &Block (() -> a) -> a+
+    # Repo URL, as expected by Git
     #
-    # @yield [String] Basename of the Tap repo directory we've just chdir'd into
-    # @return Whatever the block returns
-    def with_git_clone(&block)
-      begin
-        _git_clone
-        Dir.chdir @repo, &block
-      ensure
-        _remove_git_submodule unless @keep_submodule
-      end
-    end
+    # @return [String]
+    attr_reader :url
 
     # Overwrite the specified Formula file, in-place, on-disk
     #
-    # Haskell-y type: <tt>(String, &Block (Formula -> Formula)) -> nil</tt>
+    # A directory named +Formula+ is assumed to be on +Dir.pwd+.
     #
     # If no block is passed, then this tries to find the formula file, but then
-    # does nothing.
+    # makes no changes to that Formula.
     #
     # @param formula [String] Part of the name to the file within the +Formula+
     #     directory inside the Tap repo's directory, excluding the +.rb+ suffix.
-    # @yield [Formula]
-    # @return [Formula] as returned from the block,
-    #     assuming it obediantly returns a {Formula}.
-    def on_formula(formula, &block)
-      name = "#{formula}.rb"  # DOC
+    # @yieldparam formula [Formula] existing Formula
+    # @yieldreturn [Formula] after your modifications
+    # @return [Formula] new Formula
+    def on_formula!(formula, &block)
+      name = "#{formula}.rb"
       block ||= ->(n) { n }
-      Dir.chdir 'Formula' do  # DOC
+      Dir.chdir 'Formula' do
         File.open name, 'r' do |old_file|
           File.open "#{name}.new", 'w' do |new_file|
             new_file.write(
@@ -78,54 +64,6 @@ module HomebrewAutomation
       end
     end
 
-    # Set Git user's name and email
-    #
-    # Reads environment variables:
-    # - TRAVIS_GIT_USER_NAME
-    # - TRAVIS_GIT_USER_EMAIL
-    #
-    # If either env var is not set, do nothing.
-    def git_config
-      name = ENV['TRAVIS_GIT_USER_NAME']
-      email = ENV['TRAVIS_GIT_USER_EMAIL']
-      if name && email
-        system 'git', 'config', '--global', 'user.name', name
-        system 'git', 'config', '--global', 'user.email', email
-      end
-    end
-
-    # Just +git commit -am "$msg"+
-    #
-    # @param msg [String] Git commit message
-    # @raise [StandardError]
-    def git_commit_am(msg)
-      complain unless system "git", "commit", "-am", msg
-    end
-
-    # Just +git push+
-    #
-    # @raise [StandardError]
-    def git_push
-      complain unless system "git", "push"
-    end
-
-    # @raise [StandardError]
-    def _git_clone
-      complain unless system "git", "clone", @url
-    end
-
-    # @raise [StandardError]
-    def _remove_git_submodule
-      complain unless system "rm", "-rf", @repo
-    end
-
-    private
-
-    def complain
-      puts "HEY! Something has gone wrong and I need to complain. Stacktrace follows:"
-      puts caller
-    end
-
   end
 
 end

data/lib/homebrew_automation/version.rb

@@ -2,6 +2,6 @@
 module HomebrewAutomation
 
   # Yep.
-  VERSION = '0.1.5'
+  VERSION = '0.1.6'
 
 end

data/lib/homebrew_automation/workflow.rb

@@ -1,7 +1,6 @@
 
 require_relative './mac_os.rb'
-require_relative './bintray.rb'
-require_relative './source_dist.rb'
+require_relative './bottle.rb'
 
 module HomebrewAutomation
 
@@ -18,25 +17,45 @@ module HomebrewAutomation
     # @param tap [Tap]
     # @param formula_name [String] the name of the formula in the Tap
     # @param bversion [Bintray::Version]
+    #
+    # @param mac_os [Class] the MacOS class
+    # @param bottle [Class] the Bottle class
+    # @param keep_homebrew_tmp [Boolean] keep the HOMEBREW_TEMP directory
+    #
     # @return [Bottle]
-    def build_and_upload_bottle(sdist, tap, formula_name, bversion, keep_homebrew_tmp: false)
-      os_name = MacOS.identify_version
-      tap.with_git_clone do
-        tap.on_formula(formula_name) do |formula|
+    def build_and_upload_bottle!(
+        sdist,
+        tap,
+        git,
+        formula_name,
+        bversion,
+        mac_os: MacOS,
+        bottle: Bottle,
+        keep_tap_repo: false,
+        keep_homebrew_tmp: false)
+      os_name = mac_os.identify_version!
+      git.with_clone!(tap.url, tap.repo, keep_dir: keep_tap_repo) do |cloned_dir|
+        tap.on_formula! formula_name do |formula|
           formula.put_sdist(sdist.url, sdist.sha256)
         end
-        tap.git_commit_am "Throwaway commit; just for building bottles"
-
-        local_tap_url = File.realpath('.')
-        bottle = Bottle.new(local_tap_url, formula_name, os_name, keep_tmp: keep_homebrew_tmp)
-        bottle.build
-
-        # Bintray auto-creates Versions on file-upload.
-        # Re-creating an existing Version results in a 409.
-        #bversion.create!
-        bversion.upload_file!(bottle.filename, bottle.content)
-
-        bottle
+        git.commit_am! "Throwaway commit; just for building bottles"
+        bot = bottle.new(
+          'homebrew-automation/tmp-tap',
+          cloned_dir,
+          formula_name,
+          os_name,
+          keep_tmp: keep_homebrew_tmp)
+        bot.build! do |filename, contents|
+          # Bintray auto-creates Versions on file-upload.
+          # Re-creating an existing Version results in a 409.
+          #bversion.create!
+          begin
+            bversion.upload_file!(filename, contents)
+          rescue Bintray::Version::FileAlreadyExists
+            puts "A file with the same name as the one you're uploading already exits on Bintray"
+          end
+        end
+        bot
       end
     end
 
@@ -51,10 +70,10 @@ module HomebrewAutomation
     # @param tap [Tap]
     # @param formula_name [String] the name of the formula in the Tap
     # @param bversion [Bintray::Version]
-    # @return [Formula]
-    def gather_and_publish_bottles(sdist, tap, formula_name, bversion)
-      tap.with_git_clone do
-        tap.on_formula(formula_name) do |formula|
+    # @return [NilClass]
+    def gather_and_publish_bottles!(sdist, tap, formula_name, bversion)
+      git.with_clone!(tap.url, tap.repo) do
+        tap.on_formula! formula_name do |formula|
           bottles = bversion.gather_bottles
           bottles.reduce(
             formula.
@@ -64,10 +83,11 @@ module HomebrewAutomation
             f.put_bottle(os, checksum)
           end
         end
-        tap.git_config
-        tap.git_commit_am "Add bottles for #{formula_name}@#{bversion.version_name}"
-        tap.git_push
+        git.config!
+        git.commit_am! "Add bottles for #{formula_name}@#{bversion.version_name}"
+        git.push!
       end
+      nil
     end
 
   end

data/lib/homebrew_automation.rb

@@ -4,6 +4,7 @@
 require_relative 'homebrew_automation/bintray.rb'
 require_relative 'homebrew_automation/bottle.rb'
 require_relative 'homebrew_automation/formula.rb'
+require_relative 'homebrew_automation/git.rb'
 require_relative 'homebrew_automation/mac_os.rb'
 require_relative 'homebrew_automation/source_dist.rb'
 require_relative 'homebrew_automation/tap.rb'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: homebrew_automation
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - easoncxz
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-09-30 00:00:00.000000000 Z
+date: 2019-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -149,11 +149,13 @@ files:
 - lib/homebrew_automation/bintray/client.rb
 - lib/homebrew_automation/bintray/version.rb
 - lib/homebrew_automation/bottle.rb
+- lib/homebrew_automation/brew.rb
 - lib/homebrew_automation/cli.rb
 - lib/homebrew_automation/cli/cli.rb
 - lib/homebrew_automation/cli/formula_commands.rb
 - lib/homebrew_automation/cli/workflow_commands.rb
 - lib/homebrew_automation/formula.rb
+- lib/homebrew_automation/git.rb
 - lib/homebrew_automation/mac_os.rb
 - lib/homebrew_automation/source_dist.rb
 - lib/homebrew_automation/tap.rb