git_helper 1.3.1 → 2.0.0

data/lib/git_helper/empty_commit.rb

@@ -1,7 +1,9 @@
+require_relative './local_code.rb'
+
 module GitHelper
   class EmptyCommit
     def execute
-      system("git commit --allow-empty -m \"Empty commit\"")
+      GitHelper::LocalCode.new.empty_commit
     end
   end
 end

data/lib/git_helper/forget_local_commits.rb

@@ -0,0 +1,9 @@
+require_relative './local_code.rb'
+
+module GitHelper
+  class ForgetLocalCommits
+    def execute
+      GitHelper::LocalCode.new.forget_local_commits
+    end
+  end
+end

data/lib/git_helper/highline_cli.rb

@@ -6,6 +6,15 @@ module GitHelper
       ask('New branch name?')
     end
 
+    def process_directory_remotes?(directory)
+      answer = ask("Found git directory: #{directory}. Do you wish to proceed in updating #{directory}'s remote URLs? (y/n)")
+      answer.empty? ? true : !!(answer =~ /^y/i)
+    end
+
+    def conflicting_remote_clarification
+      ask('Found git remotes for both GitHub and GitLab. Would you like to proceed with GitLab or GitHub? (github/gitlab)').downcase
+    end
+
     def title
       ask('Title?')
     end
@@ -14,33 +23,29 @@ module GitHelper
       ask('Base branch?')
     end
 
-    def merge_request_id
-      ask('Merge Request ID?')
-    end
-
-    def pull_request_id
-      ask('Pull Request ID?')
+    def code_request_id(request_type)
+      ask("#{request_type} Request ID?")
     end
 
     def accept_autogenerated_title?(autogenerated_title)
       return false unless autogenerated_title
-      answer = ask("Accept the autogenerated merge request title '#{autogenerated_title}'? (y/n)")
-      !!(answer =~ /^y/i)
+      answer = ask("Accept the autogenerated code request title '#{autogenerated_title}'? (y/n)")
+      answer.empty? ? true : !!(answer =~ /^y/i)
     end
 
     def base_branch_default?(default_branch)
-      answer = ask("Is '#{default_branch}' the correct base branch for your new merge request? (y/n)")
-      !!(answer =~ /^y/i)
+      answer = ask("Is '#{default_branch}' the correct base branch for your new code request? (y/n)")
+      answer.empty? ? true : !!(answer =~ /^y/i)
     end
 
     def squash_merge_request?
       answer = ask('Squash merge request? (y/n)')
-      !!(answer =~ /^y/i)
+      answer.empty? ? true : !!(answer =~ /^y/i)
     end
 
     def remove_source_branch?
       answer = ask('Remove source branch after merging? (y/n)')
-      !!(answer =~ /^y/i)
+      answer.empty? ? true : !!(answer =~ /^y/i)
     end
 
     def merge_method(merge_options)
@@ -48,9 +53,9 @@ module GitHelper
       merge_options[index]
     end
 
-    def apply_template?(template_file_name)
-      answer = ask("Apply the merge request template from #{template_file_name}? (y/n)")
-      !!(answer =~ /^y/i)
+    def apply_template?(template_file_name, request_type)
+      answer = ask("Apply the #{request_type} request template from #{template_file_name}? (y/n)")
+      answer.empty? ? true : !!(answer =~ /^y/i)
     end
 
     def template_to_apply(template_options, request_type)

data/lib/git_helper/local_code.rb

@@ -1,5 +1,25 @@
 module GitHelper
   class LocalCode
+    def checkout_default
+      system("git checkout $(git symbolic-ref refs/remotes/origin/HEAD | sed \"s@^refs/remotes/origin/@@\")")
+    end
+
+    def forget_local_commits
+      system("git pull")
+      system("git reset --hard origin/HEAD")
+    end
+
+    def empty_commit
+      system("git commit --allow-empty -m \"Empty commit\"")
+    end
+
+    def clean_branches
+      system("git checkout $(git symbolic-ref refs/remotes/origin/HEAD | sed \"s@^refs/remotes/origin/@@\")")
+      system("git pull")
+      system("git fetch -p")
+      system("git branch -vv | grep \"origin/.*: gone]\" | awk '{print \$1}' | grep -v \"*\" | xargs git branch -D")
+    end
+
     def new_branch(branch_name)
       system("git pull")
       system("git branch --no-track #{branch_name}")
@@ -7,7 +27,51 @@ module GitHelper
       system("git push --set-upstream origin #{branch_name}")
     end
 
-    def name
+    def change_remote(remote_name, remote_url)
+      `git remote set-url #{remote_name} #{remote_url}`
+    end
+
+    def remotes
+      `git remote -v`.split("\n")
+    end
+
+    def remote_name(remote)
+      remote.scan(/([a-zA-z]+)/).first.first
+    end
+
+    def ssh_remote?(remote)
+      remote.scan(/(git@)/).any?
+    end
+
+    def https_remote?(remote)
+      remote.scan(/(https:\/\/)/).any?
+    end
+
+    def remote_project(remote)
+      if https_remote?(remote)
+        remote.scan(/https:\/\/[\S]+\/([\S]*).git/).first.first
+      elsif ssh_remote?(remote)
+        remote.scan(/\/([\S]*).git/).first.first
+      end
+    end
+
+    def remote_source(remote)
+      if https_remote?(remote)
+        remote.scan(/https:\/\/([a-zA-z.]+)\//).first.first
+      elsif ssh_remote?(remote)
+        remote.scan(/git@([a-zA-z.]+):/).first.first
+      end
+    end
+
+    def github_repo?
+      remotes.select { |remote| remote.include?('github') }.any?
+    end
+
+    def gitlab_project?
+      remotes.select { |remote| remote.include?('gitlab') }.any?
+    end
+
+    def project_name
       # Get the repo/project name by looking in the remote URLs for the full name
       `git remote -v`.scan(/\S[\s]*[\S]+.com[\S]{1}([\S]*).git/).first.first
     end
@@ -17,24 +81,8 @@ module GitHelper
       `git branch`.scan(/\*\s([\S]*)/).first.first
     end
 
-    def default_branch(project_name, external_client, client_type)
-      if client_type == :octokit # GitHub repository
-        external_client.repository(project_name).default_branch
-      elsif client_type == :gitlab # GitLab project
-        page_number = 1
-        counter = 1
-        branches = []
-
-        while counter > 0
-          break if default_branch = branches.select { |branch| branch.default }.first
-          page_branches = external_client.branches(project_name, page: page_number, per_page: 100)
-          branches = page_branches
-          counter = page_branches.count
-          page_number += 1
-        end
-
-        default_branch.name
-      end
+    def default_branch
+      `git symbolic-ref refs/remotes/origin/HEAD | sed "s@^refs/remotes/origin/@@" | tr -d "\n"`
     end
 
     def template_options(template_identifiers)

data/lib/git_helper/merge_request.rb

@@ -4,11 +4,20 @@ require_relative './local_code.rb'
 
 module GitHelper
   class GitLabMergeRequest
-    def create
+    attr_accessor :local_project, :local_branch, :local_code, :cli, :base_branch, :new_mr_title
+
+    def initialize(options)
+      @local_project = options[:local_project]
+      @local_branch = options[:local_branch]
+      @local_code = options[:local_code]
+      @cli = options[:cli]
+    end
+
+    def create(options)
+      @base_branch = options[:base_branch]
+      @new_mr_title = options[:new_title]
+
       begin
-        # Ask these questions right away
-        base_branch
-        new_mr_title
         options = {
           source_branch: local_branch,
           target_branch: base_branch,
@@ -36,12 +45,12 @@ module GitHelper
 
     def merge
       begin
-        # Ask these questions right away
         mr_id
-        options = {}
-        options[:should_remove_source_branch] = existing_mr.should_remove_source_branch || existing_mr.force_remove_source_branch
-        options[:squash] = existing_mr.squash
-        options[:squash_commit_message] = existing_mr.title
+        options = {
+          should_remove_source_branch: existing_mr.should_remove_source_branch || existing_mr.force_remove_source_branch,
+          squash: existing_mr.squash,
+          squash_commit_message: existing_mr.title
+        }
 
         puts "Merging merge request: #{mr_id}"
         merge = gitlab_client.accept_merge_request(local_project, mr_id, options)
@@ -51,7 +60,12 @@ module GitHelper
           merge = gitlab_client.accept_merge_request(local_project, mr_id, options)
         end
 
-        puts "Merge request successfully merged: #{merge.merge_commit_sha}"
+        if merge.merge_commit_sha.nil?
+          puts 'Could not merge merge request:'
+          puts "  #{merge.merge_error}"
+        else
+          puts "Merge request successfully merged: #{merge.merge_commit_sha}"
+        end
       rescue Gitlab::Error::MethodNotAllowed => e
         puts 'Could not merge merge request:'
         puts '  The merge request is not mergeable'
@@ -64,31 +78,36 @@ module GitHelper
       end
     end
 
-    private def local_project
-      @local_project ||= local_code.name
+    private def new_mr_body
+      @new_mr_body ||= template_name_to_apply ? local_code.read_template(template_name_to_apply) : ''
     end
 
-    private def local_branch
-      @local_branch ||= local_code.branch
-    end
+    private def template_name_to_apply
+      return @template_name_to_apply if @template_name_to_apply
+      @template_name_to_apply = nil
 
-    private def autogenerated_title
-      @autogenerated_title ||= local_code.generate_title(local_branch)
-    end
+      unless mr_template_options.empty?
+        if mr_template_options.count == 1
+          apply_single_template = cli.apply_template?(mr_template_options.first, 'merge')
+          @template_name_to_apply = mr_template_options.first if apply_single_template
+        else
+          response = cli.template_to_apply(mr_template_options, 'merge')
+          @template_name_to_apply = response unless response == 'None'
+        end
+      end
 
-    private def default_branch
-      @default_branch ||= local_code.default_branch(local_project, gitlab_client, :gitlab)
+      @template_name_to_apply
     end
 
     private def mr_template_options
       @mr_template_options ||= local_code.template_options({
-                                 nested_directory_name: "merge_request_templates",
-                                 non_nested_file_name: "merge_request_template"
+                                 nested_directory_name: 'merge_request_templates',
+                                 non_nested_file_name: 'merge_request_template'
                                })
     end
 
     private def mr_id
-      @mr_id ||= cli.merge_request_id
+      @mr_id ||= cli.code_request_id('Merge')
     end
 
     private def squash_merge_request
@@ -99,61 +118,16 @@ module GitHelper
       @remove_source_branch ||= existing_project.remove_source_branch_after_merge || cli.remove_source_branch?
     end
 
-    private def new_mr_title
-      @new_mr_title ||= if cli.accept_autogenerated_title?(autogenerated_title)
-                          autogenerated_title
-                        else
-                          cli.title
-                        end
-    end
-
-    private def base_branch
-      @base_branch ||= if cli.base_branch_default?(default_branch)
-                         default_branch
-                       else
-                         cli.base_branch
-                       end
-    end
-
-    private def new_mr_body
-      @new_mr_body ||= template_name_to_apply ? local_code.read_template(template_name_to_apply) : ''
-    end
-
-    private def template_name_to_apply
-      return @template_name_to_apply if @template_name_to_apply
-      @template_name_to_apply = nil
-
-      unless mr_template_options.empty?
-        if mr_template_options.count == 1
-          apply_single_template = cli.apply_template?(mr_template_options.first)
-          @template_name_to_apply = mr_template_options.first if apply_single_template
-        else
-          response = cli.template_to_apply(mr_template_options, 'merge')
-          @template_name_to_apply = response unless response == "None"
-        end
-      end
-
-      @template_name_to_apply
+    private def existing_project
+      @existing_project ||= gitlab_client.project(local_project)
     end
 
     private def existing_mr
       @existing_mr ||= gitlab_client.merge_request(local_project, mr_id)
     end
 
-    private def existing_project
-      @existing_project ||= gitlab_client.project(local_project)
-    end
-
     private def gitlab_client
       @gitlab_client ||= GitHelper::GitLabClient.new.client
     end
-
-    private def cli
-      @cli ||= GitHelper::HighlineCli.new
-    end
-
-    private def local_code
-      @local_code ||= GitHelper::LocalCode.new
-    end
   end
 end

data/lib/git_helper/new_branch.rb

@@ -4,17 +4,9 @@ require_relative './local_code.rb'
 module GitHelper
   class NewBranch
     def execute(new_branch_name = nil)
-      branch_name = new_branch_name || cli.new_branch_name
+      branch_name = new_branch_name || GitHelper::HighlineCli.new.new_branch_name
       puts "Attempting to create a new branch: #{branch_name}"
-      local_code.new_branch(branch_name)
-    end
-
-    private def cli
-      @cli ||= GitHelper::HighlineCli.new
-    end
-
-    private def local_code
-      @local_code ||= GitHelper::LocalCode.new
+      GitHelper::LocalCode.new.new_branch(branch_name)
     end
   end
 end

data/lib/git_helper/pull_request.rb

@@ -4,11 +4,20 @@ require_relative './local_code.rb'
 
 module GitHelper
   class GitHubPullRequest
-    def create
+    attr_accessor :local_repo, :local_branch, :local_code, :cli, :base_branch, :new_pr_title
+
+    def initialize(options)
+      @local_repo = options[:local_project]
+      @local_branch = options[:local_branch]
+      @local_code = options[:local_code]
+      @cli = options[:cli]
+    end
+
+    def create(options)
+      @base_branch = options[:base_branch]
+      @new_pr_title = options[:new_title]
+
       begin
-        # Ask these questions right away
-        base_branch
-        new_pr_title
         new_pr_body
 
         puts "Creating pull request: #{new_pr_title}"
@@ -31,7 +40,6 @@ module GitHelper
 
     def merge
       begin
-        # Ask these questions right away
         pr_id
         merge_method
 
@@ -67,31 +75,36 @@ module GitHelper
       end
     end
 
-    private def local_repo
-      @local_project ||= local_code.name
+    private def new_pr_body
+      @new_pr_body ||= template_name_to_apply ? local_code.read_template(template_name_to_apply) : ''
     end
 
-    private def local_branch
-      @local_branch ||= local_code.branch
-    end
+    private def template_name_to_apply
+      return @template_name_to_apply if @template_name_to_apply
+      @template_name_to_apply = nil
 
-    private def autogenerated_title
-      @autogenerated_title ||= local_code.generate_title(local_branch)
-    end
+      unless pr_template_options.empty?
+        if pr_template_options.count == 1
+          apply_single_template = cli.apply_template?(pr_template_options.first, 'pull')
+          @template_name_to_apply = pr_template_options.first if apply_single_template
+        else
+          response = cli.template_to_apply(pr_template_options, 'pull')
+          @template_name_to_apply = response unless response == 'None'
+        end
+      end
 
-    private def default_branch
-      @default_branch ||= local_code.default_branch(local_repo, octokit_client, :octokit)
+      @template_name_to_apply
     end
 
     private def pr_template_options
       @pr_template_options ||= local_code.template_options({
-                                 nested_directory_name: "PULL_REQUEST_TEMPLATE",
-                                 non_nested_file_name: "pull_request_template"
+                                 nested_directory_name: 'PULL_REQUEST_TEMPLATE',
+                                 non_nested_file_name: 'pull_request_template'
                                })
     end
 
     private def pr_id
-      @pr_id ||= cli.pull_request_id
+      @pr_id ||= cli.code_request_id('Pull')
     end
 
     private def merge_method
@@ -107,61 +120,16 @@ module GitHelper
       @merge_options = merge_options
     end
 
-    private def new_pr_title
-      @new_pr_title ||= if cli.accept_autogenerated_title?(autogenerated_title)
-                          autogenerated_title
-                        else
-                          cli.title
-                        end
-    end
-
-    private def base_branch
-      @base_branch ||= if cli.base_branch_default?(default_branch)
-                         default_branch
-                       else
-                         cli.base_branch
-                       end
-    end
-
-    private def new_pr_body
-      @new_pr_body ||= template_name_to_apply ? local_code.read_template(template_name_to_apply) : ''
-    end
-
-    private def template_name_to_apply
-      return @template_name_to_apply if @template_name_to_apply
-      @template_name_to_apply = nil
-
-      unless pr_template_options.empty?
-        if pr_template_options.count == 1
-          apply_single_template = cli.apply_template?(pr_template_options.first)
-          @template_name_to_apply = pr_template_options.first if apply_single_template
-        else
-          response = cli.template_to_apply(pr_template_options, 'pull')
-          @template_name_to_apply = response unless response == "None"
-        end
-      end
-
-      @template_name_to_apply
+    private def existing_project
+      @existing_project ||= octokit_client.repository(local_repo)
     end
 
     private def existing_pr
       @existing_pr ||= octokit_client.pull_request(local_repo, pr_id)
     end
 
-    private def existing_project
-      @existing_project ||= octokit_client.repository(local_repo)
-    end
-
     private def octokit_client
       @octokit_client ||= GitHelper::OctokitClient.new.client
     end
-
-    private def cli
-      @cli ||= GitHelper::HighlineCli.new
-    end
-
-    private def local_code
-      @local_code ||= GitHelper::LocalCode.new
-    end
   end
 end