Merge pull request #669 from fastlane/git_repo_cleaner

Disable a bunch of git resets

app/features/build_runner/build_runner.rb

@@ -149,7 +149,6 @@ def checkout_sha(&completion_block)
           use_global_git_mutex: false
         )
       else
-        repo.reset_hard!
         logger.debug("Pulling `master` in checkout_sha")
         repo.pull
       end
@@ -221,14 +220,8 @@ def set_build_specific_env_variable(key:, value:)
       environment_variables_set << key
     end
 
-    def reset_repo_state
-      # When we're done, clean up by resetting
-      repo.reset_hard!
-    end
-
     def post_run_action
       logger.debug("Finished running #{project.project_name} for #{sha}")
-      reset_repo_state
 
       unset_build_specific_environment_variables
     end

app/shared/models/git_repo.rb

@@ -474,9 +474,8 @@ def checkout_commit(sha: nil, repo_auth: self.repo_auth, use_global_git_mutex: t
 
         success = false
         begin
-          git.reset_hard(
-            git.gcommit(sha)
-          )
+          git.gcommit(sha)
+
           success = true
         rescue StandardError => ex
           exception_context = { sha: sha }
@@ -599,10 +598,8 @@ def fetch(use_global_git_mutex: true)
     def switch_to_fork(clone_url:, branch:, sha: nil, local_branch_name:, use_global_git_mutex: false)
       perform_block(use_global_git_mutex: use_global_git_mutex) do
         logger.debug("Switching to branch #{branch} from forked repo: #{clone_url} (pulling into #{local_branch_name})")
-        reset_hard!(use_global_git_mutex: false)
         # TODO: make sure it doesn't exist yet
         git.branch(local_branch_name)
-        reset_hard!(use_global_git_mutex: false)
 
         begin
           git.pull(clone_url, branch)