summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorConnor Shea <connor.james.shea@gmail.com>2016-05-08 15:33:34 -0600
committerConnor Shea <connor.james.shea@gmail.com>2016-05-08 15:33:34 -0600
commit9cc0937b3a41caca89fa6722149248a8f7b0a447 (patch)
tree4e1fdfcb34d69da81b993f59742a35b6bccae413 /app
parent44501820152083d231459223fe09b9d9641b7c1e (diff)
downloadgitlab-ce-9cc0937b3a41caca89fa6722149248a8f7b0a447.tar.gz
Enable the Rubocop DeprecatedClassMethods cop
This reports uses of `File.exists?` and `Dir.exists?`, which were both deprecated in Ruby and will eventually be removed in favor of `.exist?`. Also fixes all existing uses of the deprecated methods.
Diffstat (limited to 'app')
-rw-r--r--app/models/ci/build.rb2
-rw-r--r--app/models/merge_request.rb2
2 files changed, 2 insertions, 2 deletions
diff --git a/app/models/ci/build.rb b/app/models/ci/build.rb
index 4bc3a225e2c..073ec9dd89e 100644
--- a/app/models/ci/build.rb
+++ b/app/models/ci/build.rb
@@ -238,7 +238,7 @@ module Ci
end
def recreate_trace_dir
- unless Dir.exists?(dir_to_trace)
+ unless Dir.exist?(dir_to_trace)
FileUtils.mkdir_p(dir_to_trace)
end
end
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb
index 4175e1e5fba..365ccc90b40 100644
--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -544,7 +544,7 @@ class MergeRequest < ActiveRecord::Base
end
def ref_is_fetched?
- File.exists?(File.join(project.repository.path_to_repo, ref_path))
+ File.exist?(File.join(project.repository.path_to_repo, ref_path))
end
def ensure_ref_fetched