Commit e96fd232 authored by Stan Hu's avatar Stan Hu

Remove unnecessary includes of ShellAdapter

Determined by running the script:

```
included = `git grep --name-only ShellAdapter`.chomp.split("\n")
used = `git grep --name-only gitlab_shell`.chomp.split("\n")
included - used
```
parent fa3528ee
...@@ -4,6 +4,8 @@ module Storage ...@@ -4,6 +4,8 @@ module Storage
module LegacyNamespace module LegacyNamespace
extend ActiveSupport::Concern extend ActiveSupport::Concern
include Gitlab::ShellAdapter
def move_dir def move_dir
proj_with_tags = first_project_with_container_registry_tags proj_with_tags = first_project_with_container_registry_tags
......
...@@ -3,8 +3,6 @@ ...@@ -3,8 +3,6 @@
class ProjectMember < Member class ProjectMember < Member
SOURCE_TYPE = 'Project'.freeze SOURCE_TYPE = 'Project'.freeze
include Gitlab::ShellAdapter
belongs_to :project, foreign_key: 'source_id' belongs_to :project, foreign_key: 'source_id'
# Make sure project member points only to project as it source # Make sure project member points only to project as it source
......
...@@ -3,7 +3,6 @@ ...@@ -3,7 +3,6 @@
class Namespace < ActiveRecord::Base class Namespace < ActiveRecord::Base
include CacheMarkdownField include CacheMarkdownField
include Sortable include Sortable
include Gitlab::ShellAdapter
include Gitlab::VisibilityLevel include Gitlab::VisibilityLevel
include Routable include Routable
include AfterCommitQueue include AfterCommitQueue
......
# frozen_string_literal: true # frozen_string_literal: true
class ProtectedBranch < ActiveRecord::Base class ProtectedBranch < ActiveRecord::Base
include Gitlab::ShellAdapter
include ProtectedRef include ProtectedRef
protected_ref_access_levels :merge, :push protected_ref_access_levels :merge, :push
......
# frozen_string_literal: true # frozen_string_literal: true
class ProtectedTag < ActiveRecord::Base class ProtectedTag < ActiveRecord::Base
include Gitlab::ShellAdapter
include ProtectedRef include ProtectedRef
validates :name, uniqueness: { scope: :project_id } validates :name, uniqueness: { scope: :project_id }
......
...@@ -17,7 +17,6 @@ class Repository ...@@ -17,7 +17,6 @@ class Repository
#{REF_ENVIRONMENTS} #{REF_ENVIRONMENTS}
].freeze ].freeze
include Gitlab::ShellAdapter
include Gitlab::RepositoryCacheAdapter include Gitlab::RepositoryCacheAdapter
attr_accessor :full_path, :disk_path, :project, :is_wiki attr_accessor :full_path, :disk_path, :project, :is_wiki
......
...@@ -5,7 +5,6 @@ class RepositoryUpdateRemoteMirrorWorker ...@@ -5,7 +5,6 @@ class RepositoryUpdateRemoteMirrorWorker
UpdateError = Class.new(StandardError) UpdateError = Class.new(StandardError)
include ApplicationWorker include ApplicationWorker
include Gitlab::ShellAdapter
sidekiq_options retry: 3, dead: false sidekiq_options retry: 3, dead: false
......
class RenameReservedProjectNames < ActiveRecord::Migration[4.2] class RenameReservedProjectNames < ActiveRecord::Migration[4.2]
include Gitlab::Database::MigrationHelpers include Gitlab::Database::MigrationHelpers
include Gitlab::ShellAdapter
DOWNTIME = false DOWNTIME = false
......
class RenameMoreReservedProjectNames < ActiveRecord::Migration[4.2] class RenameMoreReservedProjectNames < ActiveRecord::Migration[4.2]
include Gitlab::Database::MigrationHelpers include Gitlab::Database::MigrationHelpers
include Gitlab::ShellAdapter
DOWNTIME = false DOWNTIME = false
......
...@@ -3,8 +3,6 @@ ...@@ -3,8 +3,6 @@
module Gitlab module Gitlab
module BitbucketServerImport module BitbucketServerImport
class Importer class Importer
include Gitlab::ShellAdapter
attr_reader :recover_missing_commits attr_reader :recover_missing_commits
attr_reader :project, :project_key, :repository_slug, :client, :errors, :users attr_reader :project, :project_key, :repository_slug, :client, :errors, :users
attr_accessor :logger attr_accessor :logger
......
...@@ -4,7 +4,6 @@ module Gitlab ...@@ -4,7 +4,6 @@ module Gitlab
module ImportExport module ImportExport
class RepoRestorer class RepoRestorer
include Gitlab::ImportExport::CommandLineUtil include Gitlab::ImportExport::CommandLineUtil
include Gitlab::ShellAdapter
def initialize(project:, shared:, path_to_bundle:) def initialize(project:, shared:, path_to_bundle:)
@project = project @project = project
......
...@@ -11,10 +11,6 @@ describe ProjectMember do ...@@ -11,10 +11,6 @@ describe ProjectMember do
it { is_expected.to validate_inclusion_of(:access_level).in_array(Gitlab::Access.values) } it { is_expected.to validate_inclusion_of(:access_level).in_array(Gitlab::Access.values) }
end end
describe 'modules' do
it { is_expected.to include_module(Gitlab::ShellAdapter) }
end
describe '.access_level_roles' do describe '.access_level_roles' do
it 'returns Gitlab::Access.options' do it 'returns Gitlab::Access.options' do
expect(described_class.access_level_roles).to eq(Gitlab::Access.options) expect(described_class.access_level_roles).to eq(Gitlab::Access.options)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment