summaryrefslogtreecommitdiff
path: root/scripts/lint-rugged
diff options
context:
space:
mode:
Diffstat (limited to 'scripts/lint-rugged')
-rwxr-xr-xscripts/lint-rugged6
1 files changed, 2 insertions, 4 deletions
diff --git a/scripts/lint-rugged b/scripts/lint-rugged
index 3f8fcb558e3..cabd083e9f9 100755
--- a/scripts/lint-rugged
+++ b/scripts/lint-rugged
@@ -1,7 +1,7 @@
#!/usr/bin/env ruby
ALLOWED = [
- # Can be deleted (?) once rugged is no longer used in production. Doesn't make Rugged calls.
+ # Can be fixed once Rugged is no longer used in production. Doesn't make Rugged calls.
'config/initializers/8_metrics.rb',
# Can be deleted once wiki's are fully (mandatory) migrated
@@ -13,14 +13,12 @@ ALLOWED = [
# Needs to be migrated, https://gitlab.com/gitlab-org/gitaly/issues/954
'lib/tasks/gitlab/cleanup.rake',
- # https://gitlab.com/gitlab-org/gitaly/issues/961
- 'app/models/repository.rb',
-
# The only place where Rugged code is still allowed in production
'lib/gitlab/git/'
].freeze
rugged_lines = IO.popen(%w[git grep -i -n rugged -- app config lib], &:read).lines
+rugged_lines = rugged_lines.select { |l| /^[^:]*\.rb:/ =~ l }
rugged_lines = rugged_lines.reject { |l| l.start_with?(*ALLOWED) }
rugged_lines = rugged_lines.reject do |line|
code, _comment = line.split('# ', 2)