gitlab-styles 3.2.0 → 5.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 592bf0393cc58badea566d11006eba982318048f25803f288e8616f15d9c1a56
-  data.tar.gz: ace3d41b0da57eea470c4537227bb195aee8db431b59c01c5c31a7fd559b4e2a
+  metadata.gz: 30bceb716d5e39ed5e48f4d42fd4457eb0a0608be5419e03ac325a3db385da08
+  data.tar.gz: 50058ac5c12783bd57e55e9cf19973ffb9aec21c277e3ff20839849271feb64c
 SHA512:
-  metadata.gz: f463f3d74bd39abfeba23713375eb99f8f8ed5a61b259110ef193a0692b6073fddcd89291f57712237ce7834f60414158ebfd83d9dd774a54489bdd4582043c0
-  data.tar.gz: db51862db754ac163596f6516b882a7c75c3b9822bb9c33a67fed262293b360b792fe471305848e51ecefb06f0d6f63c7d2976f01493fbba60bc6e2d79016142
+  metadata.gz: b5f700edd701ed7b465a1e5bac5971dee3e7f31df2603a03dc76fd048feb98ec23e4fb1e70b2704e5f3899eecb0c2f6817a1d508bcf1708a092587e9ba2a4f86
+  data.tar.gz: 3ba624b97b7bab08fa1fb26b82cae5f72494e639a1c7cc000d95098ad39807218e15e118654a2518076ae3409f31ee55caa980e3d88cac26394d26e10f079b0f

data/.editorconfig

@@ -0,0 +1,18 @@
+# top-most EditorConfig file
+root = true
+
+# Unix-style newlines with a newline ending every file
+[*]
+end_of_line = lf
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+[*.{rb,yml}]
+indent_size = 2
+
+[*.{rb,yml,md}]
+indent_style = space
+charset = utf-8
+
+[*.{md,markdown}]
+trim_trailing_whitespace = false

data/.gitignore

@@ -3,10 +3,12 @@
 /Gemfile.lock
 /_yardoc/
 /coverage/
-/doc/
 /pkg/
 /spec/reports/
 /tmp/
 
 # rspec failure tracking
 .rspec_status
+
+# Ignore IDE specific files
+.idea/

data/.gitlab-ci.yml

@@ -1,13 +1,34 @@
-image: ruby:2.3
+stages:
+  - test
+  - deploy
 
-before_script:
-  - bundle --version
-  - bundle install
+default:
+  image: ruby:2.6
+  tags:
+    - gitlab-org
+  before_script:
+    - bundle --version
+    - bundle install
+
+workflow:
+  rules:
+    # For merge requests, create a pipeline.
+    - if: '$CI_MERGE_REQUEST_IID'
+    # For `master` branch, create a pipeline (this includes on schedules, pushes, merges, etc.).
+    - if: '$CI_COMMIT_BRANCH == "master"'
+    # For tags, create a pipeline.
+    - if: '$CI_COMMIT_TAG'
 
 styles:
+  stage: test
   script:
     - bundle exec rubocop --debug --parallel
 
 specs:
+  stage: test
   script:
     - bundle exec rspec
+
+include:
+  - project: 'gitlab-org/quality/pipeline-common'
+    file: '/ci/gem-release.yml'

data/.gitlab/merge_request_templates/Release.md

@@ -0,0 +1,35 @@
+<!-- Replace `v4.5.0` with the previous release here, and `e18d76b309e42888759c1effe96767f13e34ae55`
+with the latest commit from https://gitlab.com/gitlab-org/gitlab-styles/commits/master that will be included in the release. -->
+- Diff: https://gitlab.com/gitlab-org/gitlab-styles/compare/v4.5.0...e18d76b309e42888759c1effe96767f13e34ae55
+
+- Release notes:
+
+<!-- Keep the sections order but remove the empty sections -->
+
+```markdown
+### New features and features updates
+
+- !aaa <Title of the aaa MR>.
+
+### Fixes
+
+- !bbb <Title of the bbb MR>.
+
+### Doc changes
+
+- !ccc <Title of the ccc MR>.
+
+### Other changes (tooling, technical debt)
+
+- !ddd <Title of the ddd MR>.
+```
+
+- Checklist before merging:
+  - [ ] Diff link is up-to-date.
+  - [ ] Based on the diff, `lib/gitlab/styles/version.rb` is updated, according to [SemVer](https://semver.org).
+  - [ ] Release notes are accurate.
+
+- Checklist after merging:
+  - [ ] [Update the release notes for the newly created tag](docs/release_process.md#how-to).
+
+/label ~"Engineering Productivity" ~"feature" ~"feature::maintenance" ~"static code analysis"

data/.rubocop.yml

@@ -1,3 +1,5 @@
 inherit_from:
   - rubocop-default.yml
-  - .rubocop_todo.yml
+
+require:
+  - rubocop/cop/internal_affairs

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in gitlab-rubocop.gemspec
@@ -5,6 +7,7 @@ gemspec
 
 group :test do
   # Pin these dependencies, otherwise a new rule could break the CI pipelines
-  gem 'rubocop', '0.74.0'
-  gem 'rubocop-rspec', '1.36.0'
+  gem 'rubocop', '0.89.1'
+  gem 'rubocop-rspec', '1.44.1'
+  gem 'rspec-parameterized', '0.4.2', require: false
 end

data/README.md

@@ -42,6 +42,7 @@ rules:
 - `rubocop-gemspec.yml`
 - `rubocop-layout.yml`
 - `rubocop-lint.yml`
+- `rubocop-migrations.yml`
 - `rubocop-metrics.yml`
 - `rubocop-naming.yml`
 - `rubocop-performance.yml`
@@ -77,6 +78,10 @@ https://gitlab.com/gitlab-org/gitlab-styles. This project is intended to be a
 safe, welcoming space for collaboration, and contributors are expected to adhere
 to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.
 
+## Release Process
+
+Please refer to the [Release Process](docs/release_process.md).
+
 ## License
 
 The gem is available as open source under the terms of the

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 

data/gitlab-styles.gemspec

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'gitlab/styles/version'
 
 Gem::Specification.new do |spec|
+  spec.required_ruby_version = '>= 2.6'
   spec.name          = 'gitlab-styles'
   spec.version       = Gitlab::Styles::VERSION
   spec.authors       = ['GitLab']
@@ -13,17 +16,17 @@ Gem::Specification.new do |spec|
   spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0").reject do |f|
-    f.match(%r{^(test|spec|features)/})
+    f.match(%r{^(docs|test|spec|features)/})
   end
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'rubocop', '~> 0.74.0'
+  spec.add_dependency 'rubocop', '~> 0.89.1'
   spec.add_dependency 'rubocop-gitlab-security', '~> 0.1.0'
-  spec.add_dependency 'rubocop-performance', '~> 1.4.1'
-  spec.add_dependency 'rubocop-rails', '~> 2.0'
-  spec.add_dependency 'rubocop-rspec', '~> 1.36'
+  spec.add_dependency 'rubocop-performance', '~> 1.8.1'
+  spec.add_dependency 'rubocop-rails', '~> 2.8'
+  spec.add_dependency 'rubocop-rspec', '~> 1.44'
 
   spec.add_development_dependency 'bundler', '~> 1.16'
   spec.add_development_dependency 'rake', '~> 10.0'

data/lib/gitlab/styles.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gitlab/styles/version'
 
 module Gitlab

data/lib/gitlab/styles/rubocop.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gitlab/styles/rubocop/cop/custom_error_class'
 require 'gitlab/styles/rubocop/cop/gem_fetcher'
 require 'gitlab/styles/rubocop/cop/active_record_serialize'
@@ -6,10 +8,15 @@ require 'gitlab/styles/rubocop/cop/polymorphic_associations'
 require 'gitlab/styles/rubocop/cop/active_record_dependent'
 require 'gitlab/styles/rubocop/cop/in_batches'
 require 'gitlab/styles/rubocop/cop/line_break_after_guard_clauses'
+require 'gitlab/styles/rubocop/cop/line_break_around_conditional_block'
+require 'gitlab/styles/rubocop/cop/code_reuse/active_record'
+require 'gitlab/styles/rubocop/cop/migration/update_large_table'
 require 'gitlab/styles/rubocop/cop/without_reactive_cache'
 require 'gitlab/styles/rubocop/cop/rspec/single_line_hook'
 require 'gitlab/styles/rubocop/cop/rspec/have_link_parameters'
 require 'gitlab/styles/rubocop/cop/rspec/verbose_include_metadata'
+require 'gitlab/styles/rubocop/cop/rspec/empty_line_after_shared_example'
+require 'gitlab/styles/rubocop/cop/rspec/empty_line_after_let_block'
 
 module Gitlab
   module Styles

data/lib/gitlab/styles/rubocop/cop/active_record_dependent.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../model_helpers'
 
 module Gitlab
@@ -9,7 +11,7 @@ module Gitlab
           include ModelHelpers
 
           MSG = 'Do not use `dependent: to remove associated data, ' \
-            'use foreign keys with cascading deletes instead'.freeze
+            'use foreign keys with cascading deletes instead'
 
           METHOD_NAMES = [:has_many, :has_one, :belongs_to].freeze
 
@@ -20,7 +22,7 @@ module Gitlab
             node.children.last.each_node(:pair) do |pair|
               key_name = pair.children[0].children[0]
 
-              add_offense(pair, location: :expression) if key_name == :dependent
+              add_offense(pair) if key_name == :dependent
             end
           end
         end

data/lib/gitlab/styles/rubocop/cop/active_record_serialize.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../model_helpers'
 
 module Gitlab
@@ -8,7 +10,7 @@ module Gitlab
         class ActiveRecordSerialize < RuboCop::Cop::Cop
           include ModelHelpers
 
-          MSG = 'Do not store serialized data in the database, use separate columns and/or tables instead'.freeze
+          MSG = 'Do not store serialized data in the database, use separate columns and/or tables instead'
 
           def on_send(node)
             return unless in_model?(node)

data/lib/gitlab/styles/rubocop/cop/code_reuse/active_record.rb

@@ -0,0 +1,131 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Styles
+    module Rubocop
+      module Cop
+        module CodeReuse
+          # Cop that denies the use of ActiveRecord methods outside of models.
+          class ActiveRecord < RuboCop::Cop::Cop
+            MSG = 'This method can only be used inside an ActiveRecord model: ' \
+        'https://gitlab.com/gitlab-org/gitlab-foss/issues/49653'
+
+            # Various methods from ActiveRecord::Querying that are denied. We
+            # exclude some generic ones such as `any?` and `first`, as these may
+            # lead to too many false positives, since `Array` also supports these
+            # methods.
+            #
+            # The keys of this Hash are the denied method names. The values are
+            # booleans that indicate if the method should only be denied if any
+            # arguments are provided.
+            NOT_ALLOWED = {
+              average: true,
+              calculate: true,
+              count_by_sql: true,
+              create_with: true,
+              distinct: false,
+              eager_load: true,
+              exists?: true,
+              find_by: true,
+              find_by!: true,
+              find_by_sql: true,
+              find_each: true,
+              find_in_batches: true,
+              find_or_create_by: true,
+              find_or_create_by!: true,
+              find_or_initialize_by: true,
+              first!: false,
+              first_or_create: true,
+              first_or_create!: true,
+              first_or_initialize: true,
+              from: true,
+              group: true,
+              having: true,
+              ids: false,
+              includes: true,
+              joins: true,
+              limit: true,
+              lock: false,
+              many?: false,
+              offset: true,
+              order: true,
+              pluck: true,
+              preload: true,
+              readonly: false,
+              references: true,
+              reorder: true,
+              rewhere: true,
+              take: false,
+              take!: false,
+              unscope: false,
+              where: false,
+              with: true
+            }.freeze
+
+            def on_send(node)
+              receiver = node.children[0]
+              send_name = node.children[1]
+              first_arg = node.children[2]
+
+              return unless receiver && NOT_ALLOWED.key?(send_name)
+
+              # If the rule requires an argument to be given, but none are
+              # provided, we won't register an offense. This prevents us from
+              # adding offenses for `project.group`, while still covering
+              # `Project.group(:name)`.
+              return if NOT_ALLOWED[send_name] && !first_arg
+
+              add_offense(node, location: :selector)
+            end
+
+            # We can not auto correct code like this, as it requires manual
+            # refactoring. Instead, we'll just allow the surrounding scope.
+            #
+            # Despite this method's presence, you should not use it. This method
+            # exists to make it possible to allow large chunks of offenses we
+            # can't fix in the short term. If you are writing new code, follow the
+            # code reuse guidelines, instead of allowing any new offenses.
+            def autocorrect(node)
+              scope = surrounding_scope_of(node)
+              indent = indentation_of(scope)
+
+              lambda do |corrector|
+                # This prevents us from inserting the same enable/disable comment
+                # for a method or block that has multiple offenses.
+                next if allowed_scopes.include?(scope)
+
+                corrector.insert_before(
+                  scope.source_range,
+                  "# rubocop: disable #{cop_name}\n#{indent}"
+                )
+
+                corrector.insert_after(
+                  scope.source_range,
+                  "\n#{indent}# rubocop: enable #{cop_name}"
+                )
+
+                allowed_scopes << scope
+              end
+            end
+
+            def indentation_of(node)
+              ' ' * node.loc.expression.source_line[/\A */].length
+            end
+
+            def surrounding_scope_of(node)
+              %i[def defs block begin].each do |type|
+                if (found = node.each_ancestor(type).first)
+                  return found
+                end
+              end
+            end
+
+            def allowed_scopes
+              @allowed_scopes ||= Set.new
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/styles/rubocop/cop/custom_error_class.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Gitlab
   module Styles
     module Rubocop
@@ -16,10 +18,11 @@ module Gitlab
         #   # good
         #   FooError = Class.new(StandardError)
         class CustomErrorClass < RuboCop::Cop::Cop
-          MSG = 'Use `Class.new(SuperClass)` to define an empty custom error class.'.freeze
+          MSG = 'Use `Class.new(SuperClass)` to define an empty custom error class.'
 
           def on_class(node)
-            _klass, parent, body = node.children
+            parent = node.parent_class
+            body = node.body
 
             return if body
 
@@ -27,11 +30,13 @@ module Gitlab
 
             return unless parent_klass&.to_s&.end_with?('Error')
 
-            add_offense(node, location: :expression)
+            add_offense(node)
           end
 
           def autocorrect(node)
-            klass, parent, _body = node.children
+            klass = node.identifier
+            parent = node.parent_class
+
             replacement = "#{class_name_from_node(klass)} = Class.new(#{class_name_from_node(parent)})"
 
             lambda do |corrector|

data/lib/gitlab/styles/rubocop/cop/gem_fetcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Gitlab
   module Styles
     module Rubocop
@@ -6,7 +8,7 @@ module Gitlab
         # `Gemfile` in order to avoid additional points of failure beyond
         # rubygems.org.
         class GemFetcher < RuboCop::Cop::Cop
-          MSG = 'Do not use gems from git repositories, only use gems from RubyGems.'.freeze
+          MSG = 'Do not use gems from git repositories, only use gems from RubyGems.'
 
           GIT_KEYS = [:git, :github].freeze
 
@@ -18,7 +20,7 @@ module Gitlab
 
             node.children.last.each_node(:pair) do |pair|
               key_name = pair.children[0].children[0].to_sym
-              add_offense(node, location: pair.source_range, message: MSG) if GIT_KEYS.include?(key_name)
+              add_offense(node, location: pair.source_range) if GIT_KEYS.include?(key_name)
             end
           end