diff options
| author | Rémy Coutable <remy@rymai.me> | 2018-01-25 20:52:01 +0300 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2018-01-25 20:52:01 +0300 |
| commit | 26c4dea7ad37f5680ada812e212274ebd8473a78 (patch) | |
| tree | e94cbb24402a46947996ae1783995d9f11ae5524 /config | |
| parent | e4c2ce6f67ffdaa88ee394c93e07f175f6e309eb (diff) | |
| parent | 6d6f7536bd9b5bcbf94dfbe15cc86e84d06527f5 (diff) | |
Merge branch 'lint-rugged' into 'master'
Add a lint check to restrict use of Rugged
See merge request gitlab-org/gitlab-ce!16656
Diffstat (limited to 'config')
| -rw-r--r-- | config/initializers/rugged_use_gitlab_git_attributes.rb | 28 |
1 files changed, 0 insertions, 28 deletions
diff --git a/config/initializers/rugged_use_gitlab_git_attributes.rb b/config/initializers/rugged_use_gitlab_git_attributes.rb deleted file mode 100644 index c0d45caec42..00000000000 --- a/config/initializers/rugged_use_gitlab_git_attributes.rb +++ /dev/null @@ -1,28 +0,0 @@ -# We don't want to ever call Rugged::Repository#fetch_attributes, because it has -# a lot of I/O overhead: -# <https://gitlab.com/gitlab-org/gitlab_git/commit/340e111e040ae847b614d35b4d3173ec48329015> -# -# While we don't do this from within the GitLab source itself, the Linguist gem -# has a dependency on Rugged and uses the gitattributes file when calculating -# repository-wide language statistics: -# <https://github.com/github/linguist/blob/v4.7.0/lib/linguist/lazy_blob.rb#L33-L36> -# -# The options passed by Linguist are those assumed by Gitlab::Git::InfoAttributes -# anyway, and there is no great efficiency gain from just fetching the listed -# attributes with our implementation, so we ignore the additional arguments. -# -module Rugged - class Repository - module UseGitlabGitAttributes - def fetch_attributes(name, *) - attributes.attributes(name) - end - - def attributes - @attributes ||= Gitlab::Git::InfoAttributes.new(path) - end - end - - prepend UseGitlabGitAttributes - end -end |