gitlab-dangerfiles 4.11.1 → 4.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 24b2dd1bfef63a377bc72132ed9f7fdf1fc12a1f9eb57527512ea83fa6f68d73
-  data.tar.gz: 793b4ee608a320c9e89bc12caa2816485edfbcfeb5714aff67b008901a95b8b0
+  metadata.gz: 90d11c90b2540e75f8664f62f9e8dca94f3a1ad9da4454af946d578ba5d06bd4
+  data.tar.gz: c3319c3df8c5bdc2fd9a9f1a409da791615b516272872eaf06f4fc1b1f0a6f73
 SHA512:
-  metadata.gz: 004c62d5d08af0eb4af16ca487ec517916b52c8a53273175aa2eb474f5ce4c309e1bfea609b5f6ed30c5ab8fcc8f17c19dc72c366d72055d3dfa6910dc372d7f
-  data.tar.gz: a6482e431b12aa645e144406f5367a529e7d1a6b10d4963ac2b715499cf0b11028328c5b28fe4ba82f65704b49857d9a5b62856d9f7de58776cab1965a421ccd
+  metadata.gz: 943f6712c05394b8476f22eb937ebc5267c570e64e4ef2eb9414404353d92ff69dae10dfece405e6d2cbb7b4f117c873ec3b57998d5cf3db620ad8de239dc931
+  data.tar.gz: fd05d33d287a9d6661bd8cadc59613e9ad717427c22dfd97ed14be94cfa90c71b8f0d46b43a4135a07348acf3d173f4fe38183e5164e5afb4cca192630c8eb87

data/.gitlab/CODEOWNERS

@@ -1 +1 @@
-* @gl-dx/eng-prod
+* @gl-dx

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab-dangerfiles (4.11.1)
+    gitlab-dangerfiles (4.13.0)
       danger (>= 9.3.0)
       danger-gitlab (>= 8.0.0)
       rake (~> 13.0)

data/README.md

@@ -169,16 +169,10 @@ This rule ensures the merge request isn't too big to be reviewed, otherwise it s
 
 #### `commits_counter`
 
-This rule posts a failure if the merge request has more than 20 commits.
+This rule posts a warning if the merge request has more than 20 commits.
 
-#### `danger_code_review`
-
-This rule posts a warning/failure when `@GitLabDuo` isn't assigned as a reviewer.
-
-##### Available configurations
-
-- `duo_code_review`: Whether a review from GitLab Duo Code is `:mandatory` or `:optional`.
-  Default to `:optional`.
+For merge requests in [`gitlab-org/gitlab`](https://gitlab.com/gitlab-org/gitlab), it posts a failure
+instead, unless the merge request is set to squash commits.
 
 #### `metadata`
 
@@ -225,6 +219,32 @@ helper.config.auto_assign_for_roulette_roles = [:reviewer]
 helper.config.auto_assign_for_roulette_labels = ["maintenance::dependency"]
 ```
 
+##### CODEOWNERS boost
+
+By default, the roulette selects reviewers and maintainers purely based on category membership and
+capacity weights. You can optionally give CODEOWNERS-eligible approvers additional weight in the
+selection pool, making them more likely to be picked without hard-gating on them.
+
+Configure `codeowners_reviewer_boost_for_roulette` and/or `codeowners_maintainer_boost_for_roulette`
+with a positive integer to add that many extra entries in the weighted pool for matching reviewers
+or maintainers. The boost stacks on top of existing capacity, hungry, and traintainer weights.
+
+```ruby
+# Prefer code owners for review, but not for maintainer approval
+helper.config.codeowners_reviewer_boost_for_roulette = 3
+
+# Prefer code owners for both review and maintainer approval
+helper.config.codeowners_reviewer_boost_for_roulette = 3
+helper.config.codeowners_maintainer_boost_for_roulette = 2
+```
+
+| Value | Behavior |
+|-------|----------|
+| `0` (default) | No change — existing behavior preserved |
+| `1-2` | Mild preference for code owners |
+| `3` | Recommended — meaningful preference without starving non-owners |
+| `4+` | Strong preference — use with caution on small teams |
+
 ##### Available configurations
 
 Note: These options aren't actually used in the `simple_roulette` rule, but can be used in your own
@@ -253,14 +273,12 @@ In order to run `danger` on GitLab CI, perform the following steps:
 
 1. If not yet done, create a `Dangerfile` at the top-level of your project. Refer to [Usage](#usage) to
    see how to set it up.
-2. In `.gitlab-ci.yml`, include [CI configuration](https://gitlab.com/gitlab-org/quality/pipeline-common/-/blob/master/ci/danger-review.yml)
-   which defines `danger-review` [CI job](https://docs.gitlab.com/ee/ci/jobs/):
+2. In `.gitlab-ci.yml`, include the [`danger-review` CI/CD component](https://gitlab.com/gitlab-org/components/danger-review)
+   which defines the `danger-review` [CI job](https://docs.gitlab.com/ee/ci/jobs/):
 
 ```yaml
 include:
-  - project: 'gitlab-org/quality/pipeline-common'
-    file:
-      - '/ci/danger-review.yml'
+  - component: gitlab.com/gitlab-org/components/danger-review/danger-review@~latest
 ```
 
 3. Create a [Project or group access token](https://docs.gitlab.com/ee/user/project/settings/project_access_tokens.html)
@@ -277,22 +295,17 @@ its dependencies. This requires a project to have a `Gemfile` and
 `Gemfile.lock` commited. This is helpful especially if Danger is also used
 locally - with `lefthook`, for example.
 
-In order to skip Ruby and `bundler` dependency in a project, use `bundle` commands directly
-in the CI configuration:
+You don't need a `Gemfile` to use the `danger-review` component. If your project
+has no `Gemfile`, the component creates one and adds the latest `gitlab-dangerfiles`
+to it before it runs Danger.
+
+To pin the `gitlab-dangerfiles` version instead, set the `gitlab_dangerfiles_version` input:
 
 ```yaml
 include:
-  - project: 'gitlab-org/quality/pipeline-common'
-    file: '/ci/danger-review.yml'
-
-danger-review:
-  before_script:
-    - bundle init
-    # For latest version
-    - bundle add gitlab-dangerfiles
-    # OR
-    # For a pinned version
-    - bundle add gitlab-dangerfiles --version 3.1.0
+  - component: gitlab.com/gitlab-org/components/danger-review/danger-review@~latest
+    inputs:
+      gitlab_dangerfiles_version: "4.12.0"
 ```
 
 ## Local Danger Rake task

data/lib/danger/plugins/changelog.rb

@@ -198,7 +198,7 @@ module Danger
         check_result.warning("This MR has a Changelog commit for EE, but no code changes in `ee/`. Consider removing the `EE: true` trailer from your commits.")
       end
 
-      if ee_changes.any? && ee_changelog? && required_reasons.include?(:db_changes)
+      if ee_changes.any? && ee_changelog? && required_reasons.include?(:db_changes) && non_ee_database_migration_changes?
         check_result.warning("This MR has a Changelog commit with the `EE: true` trailer, but there are database changes which [requires](https://docs.gitlab.com/ee/development/changelog.html#what-warrants-a-changelog-entry) the Changelog commit to not have the `EE: true` trailer. Consider removing the `EE: true` trailer from your commits.")
       end
 
@@ -216,6 +216,12 @@ module Danger
       required_reasons.any?
     end
 
+    def non_ee_database_migration_changes?
+      helper.changes.added.by_category(:migration).any? do |change|
+        !change.file.start_with?("ee/db/")
+      end
+    end
+
     def optional?
       categories_need_changelog? && mr_without_no_changelog_label?
     end

data/lib/danger/plugins/commit_messages.rb

@@ -102,7 +102,10 @@ module Danger
       commit_linters = commits.map { |commit| lint_commit(commit) }
 
       if helper.squash_mr?
-        multi_line_commit_linter = commit_linters.detect do |commit_linter|
+        # `commits` (and therefore `commit_linters`) is ordered newest-to-oldest, matching `git log`.
+        # We want the oldest multi-line, non-merge commit, since that's the commit whose message is
+        # expected to survive the squash, so we search in reverse order.
+        multi_line_commit_linter = commit_linters.reverse.detect do |commit_linter|
           !commit_linter.merge? && commit_linter.multi_line?
         end
 

data/lib/danger/plugins/roulette.rb

@@ -53,12 +53,19 @@ module Danger
       project = (project || config_project_name).downcase
       categories = categories.map { |category| category&.downcase || :none }
 
+      reviewer_boost = helper.config.codeowners_reviewer_boost_for_roulette
+      maintainer_boost = helper.config.codeowners_maintainer_boost_for_roulette
+      co_usernames = reviewer_boost > 0 || maintainer_boost > 0 ? codeowners_eligible_usernames : Set.new
+
       Gitlab::Dangerfiles::Spinner.new(
         project: project,
         author: helper.mr_author, team_author: team_mr_author,
         labels: labels, categories: categories, random: random,
         ux_fallback_wider_community_reviewer:
-          ux_fallback_wider_community_reviewer)
+          ux_fallback_wider_community_reviewer,
+        codeowners_usernames: co_usernames,
+        codeowners_reviewer_boost: reviewer_boost,
+        codeowners_maintainer_boost: maintainer_boost)
         .spin
     end
 
@@ -120,10 +127,25 @@ module Danger
     private
 
     def ux_fallback_reviewer
-      teammates = %w[pedroms annabeldunstone seggenberger jmiocene clavimoniere]
+      teammates = %w[pedroms seggenberger clavimoniere]
       Gitlab::Dangerfiles::Teammate.find_member(teammates.sample)
     end
 
+    # Collect all CODEOWNERS-eligible approver usernames across qualifying approval rules.
+    #
+    # @return [Set<String>]
+    def codeowners_eligible_usernames
+      approval_rules = helper.mr_approval_state["rules"]
+
+      return Set.new unless approval_rules
+
+      unique_approval_rules(approval_rules).each_with_object(Set.new) do |rule, usernames|
+        next unless spin_for_approval_rule?(rule)
+
+        rule["eligible_approvers"].each { |approver| usernames << approver["username"] }
+      end
+    end
+
     def spin_for_approval_rule?(rule)
       rule["rule_type"] == "code_owner" &&
         should_include_codeowners_rule?(rule) &&

data/lib/gitlab/dangerfiles/approval.rb

@@ -4,7 +4,7 @@ require_relative "spin"
 
 module Gitlab
   module Dangerfiles
-    Approval = Struct.new(:category, :spin) do
+    Approval = Struct.new(:category, :spin, :name) do
       def self.from_approval_rule(rule, maintainer)
         category =
           if rule["section"] == "codeowners"
@@ -15,7 +15,7 @@ module Gitlab
 
         spin = Spin.new(category, nil, maintainer, :reviewer)
 
-        new(category, spin)
+        new(category, spin, rule["name"])
       end
     end
   end

data/lib/gitlab/dangerfiles/capability.rb

@@ -13,7 +13,6 @@ module Gitlab
       def self.category_to_class
         @category_to_class ||= {
           none: None,
-          test: Test,
           tooling: Tooling,
           ux: UX
         }.freeze
@@ -36,16 +35,6 @@ module Gitlab
         end
       end
 
-      class Test < Capability
-        def has_capability?(teammate)
-          return false if kind != :reviewer
-
-          area = teammate.role[/Software Engineer in Test(?:.*?, (\w+))/, 1]
-
-          !!area && labels.any?("devops::#{area.downcase}")
-        end
-      end
-
       class Tooling < Capability
         def has_capability?(teammate)
           if super

data/lib/gitlab/dangerfiles/commit_linter.rb

@@ -46,7 +46,7 @@ module Gitlab
       end
 
       def merge?
-        commit.message.start_with?("Merge branch")
+        commit.message.start_with?("Merge branch", "Merge remote-tracking branch")
       end
 
       def revert?

data/lib/gitlab/dangerfiles/config.rb

@@ -40,11 +40,6 @@ module Gitlab
       #   @return [Integer] the maximum number of allowed non-squashed/non-fixup commits for a given MR. A warning is triggered if the MR has more commits.
       attr_accessor :max_commits_count
 
-      # Rule: duo_code_review
-      # @!attribute duo_code_review
-      #   @return [Symbol] whether a review from GitLab Duo Code is `:mandatory` or `:optional`. Default to `:optional`.
-      attr_accessor :duo_code_review
-
       # Rule: simple_roulette
       # @!attribute included_optional_codeowners_sections_for_roulette
       #   @return [Array] indicating which optional codeowners sections should be included in roulette. Default to `[]`.
@@ -55,6 +50,16 @@ module Gitlab
       #   @return [Array] indicating which required codeowners sections should be excluded from roulette. Default to `[]`.
       attr_accessor :excluded_required_codeowners_sections_for_roulette
 
+      # Rule: simple_roulette
+      # @!attribute codeowners_reviewer_boost_for_roulette
+      #   @return [Integer] extra weight given to reviewers who are CODEOWNERS-eligible approvers for the current merge request. Set to 0 to disable. Default to `0`.
+      attr_accessor :codeowners_reviewer_boost_for_roulette
+
+      # Rule: simple_roulette
+      # @!attribute codeowners_maintainer_boost_for_roulette
+      #   @return [Integer] extra weight given to maintainers who are CODEOWNERS-eligible approvers for the current merge request. Set to 0 to disable. Default to `0`.
+      attr_accessor :codeowners_maintainer_boost_for_roulette
+
       # @!attribute auto_assign_for_roulette_roles
       #   @return [Array<Symbol>] which roles to auto assign as reviewers, given roulette recommendations (:reviewer, :maintainer, or both). If empty, auto-assignment is disabled. Default to `[]`.
       attr_accessor :auto_assign_for_roulette_roles
@@ -77,9 +82,10 @@ module Gitlab
         @disabled_roulette_categories = []
         @included_optional_codeowners_sections_for_roulette = []
         @excluded_required_codeowners_sections_for_roulette = []
+        @codeowners_reviewer_boost_for_roulette = 0
+        @codeowners_maintainer_boost_for_roulette = 0
         @auto_assign_for_roulette_roles = []
         @auto_assign_for_roulette_labels = []
-        @duo_code_review = :optional
       end
     end
   end

data/lib/gitlab/dangerfiles/spec_helper.rb

@@ -154,17 +154,6 @@ RSpec.shared_context "with teammates" do
     )
   end
 
-  let(:software_engineer_in_test) do
-    Gitlab::Dangerfiles::Teammate.new(
-      "username" => "software-engineer-in-test",
-      "name" => "Software Engineer in Test",
-      "role" => "Software Engineer in Test, Create:Source Code",
-      "projects" => { "gitlab" => "maintainer qa", "gitlab-qa" => "maintainer" },
-      "available" => true,
-      "tz_offset_hours" => 2.0
-    )
-  end
-
   let(:tooling_reviewer) do
     Gitlab::Dangerfiles::Teammate.new(
       "username" => "eng-prod-reviewer",
@@ -227,7 +216,6 @@ RSpec.shared_context "with teammates" do
       frontend_maintainer.to_h,
       frontend_reviewer.to_h,
       ux_reviewer.to_h,
-      software_engineer_in_test.to_h,
       tooling_reviewer.to_h,
       ci_template_reviewer.to_h,
       workhorse_reviewer.to_h,

data/lib/gitlab/dangerfiles/spinner.rb

@@ -10,9 +10,14 @@ module Gitlab
     class Spinner
       attr_reader :project, :author, :team_author, :labels, :categories
 
-      def initialize(
+      # @param codeowners_usernames [Set<String>] usernames of CODEOWNERS-eligible approvers for the MR
+      # @param codeowners_reviewer_boost [Integer] extra weight entries for CODEOWNERS-eligible reviewers
+      # @param codeowners_maintainer_boost [Integer] extra weight entries for CODEOWNERS-eligible maintainers
+      def initialize( # rubocop:disable Metrics/ParameterLists -- CODEOWNERS boost params extend the existing signature
         project:, author:, team_author: nil, labels: [], categories: [],
-        random: Random.new, ux_fallback_wider_community_reviewer: nil)
+        random: Random.new, ux_fallback_wider_community_reviewer: nil,
+        codeowners_usernames: Set.new, codeowners_reviewer_boost: 0,
+        codeowners_maintainer_boost: 0)
         @project = project
         @author = author
         @team_author = team_author
@@ -20,9 +25,12 @@ module Gitlab
         @categories = categories
         @random = random
         @ux_fallback_wider_community_reviewer = ux_fallback_wider_community_reviewer
+        @codeowners_usernames = codeowners_usernames
+        @codeowners_reviewer_boost = codeowners_reviewer_boost
+        @codeowners_maintainer_boost = codeowners_maintainer_boost
       end
 
-      # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+      # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
       def spin
         spins = categories.sort_by(&:to_s).map do |category|
           spin_for_category(category)
@@ -33,10 +41,7 @@ module Gitlab
 
         spins.each do |spin|
           case spin.category
-          when :qa
-            spin.optional_role = :maintainer if
-              categories.size > 1 && author_no_qa_capability?
-          when :test
+          when :qa, :test
             spin.optional_role = :maintainer
 
             if spin.no_reviewer?
@@ -61,8 +66,6 @@ module Gitlab
               spin.maintainer = backend_spin&.maintainer || frontend_spin&.maintainer || spin_for_category(:backend).maintainer
             end
           when :ux
-            spin.optional_role = :maintainer
-
             # We want at least a UX reviewer who can review any wider community
             # contribution even without a team designer. We assign this to Pedro.
             spin.reviewer = ux_fallback_wider_community_reviewer if
@@ -74,7 +77,7 @@ module Gitlab
 
         spins
       end
-      # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+      # rubocop:enable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
 
       # Spin a reviewer for a particular approval rule
       #
@@ -92,12 +95,8 @@ module Gitlab
 
       private
 
-      attr_reader :random, :ux_fallback_wider_community_reviewer
-
-      # MR includes QA changes, but also other changes, and author isn't an SET
-      def author_no_qa_capability?
-        !(team_author && team_author.capabilities(project).any? { |capability| capability.end_with?("qa") })
-      end
+      attr_reader :random, :ux_fallback_wider_community_reviewer,
+        :codeowners_usernames, :codeowners_reviewer_boost, :codeowners_maintainer_boost
 
       def spin_for_category(category)
         reviewers, traintainers, maintainers =
@@ -105,8 +104,16 @@ module Gitlab
             spin_role_for_category(role, category)
           end
 
-        weighted_reviewers = Weightage::Reviewers.new(reviewers, traintainers).execute
-        weighted_maintainers = Weightage::Maintainers.new(maintainers).execute
+        weighted_reviewers = Weightage::Reviewers.new(
+          reviewers, traintainers,
+          codeowners_usernames: codeowners_usernames,
+          codeowners_boost: codeowners_reviewer_boost
+        ).execute
+        weighted_maintainers = Weightage::Maintainers.new(
+          maintainers,
+          codeowners_usernames: codeowners_usernames,
+          codeowners_boost: codeowners_maintainer_boost
+        ).execute
 
         reviewer = spin_for_person(weighted_reviewers)
         maintainer = spin_for_person(weighted_maintainers)

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "4.11.1"
+    VERSION = "4.13.0"
   end
 end

data/lib/gitlab/dangerfiles/weightage/maintainers.rb

@@ -7,8 +7,13 @@ module Gitlab
     module Weightage
       # @api private
       class Maintainers
-        def initialize(maintainers)
+        # @param maintainers [Array<Gitlab::Dangerfiles::Teammate>]
+        # @param codeowners_usernames [Set<String>] usernames of CODEOWNERS-eligible approvers for the MR
+        # @param codeowners_boost [Integer] extra weight entries for CODEOWNERS-eligible maintainers
+        def initialize(maintainers, codeowners_usernames: Set.new, codeowners_boost: 0)
           @maintainers = maintainers
+          @codeowners_usernames = codeowners_usernames
+          @codeowners_boost = codeowners_boost
         end
 
         def execute
@@ -19,7 +24,7 @@ module Gitlab
 
         private
 
-        attr_reader :maintainers
+        attr_reader :maintainers, :codeowners_usernames, :codeowners_boost
 
         def add_weighted_reviewer(reviewers, reviewer, weight)
           if reviewer.reduced_capacity
@@ -27,6 +32,8 @@ module Gitlab
           else
             reviewers.fill(reviewer, reviewers.size, weight * Gitlab::Dangerfiles::Weightage::CAPACITY_MULTIPLIER)
           end
+
+          reviewers.fill(reviewer, reviewers.size, codeowners_boost) if codeowners_boost.positive? && codeowners_usernames.include?(reviewer.username)
         end
       end
     end

data/lib/gitlab/dangerfiles/weightage/reviewers.rb

@@ -18,20 +18,27 @@ module Gitlab
       # | hungry traintainer           |                              6 |
       # +------------------------------+--------------------------------+
       #
+      # When +codeowners_boost+ is configured (> 0), reviewers whose
+      # username appears in +codeowners_usernames+ receive that many
+      # additional entries in the pool on top of their base weight.
+      #
       # @api private
       class Reviewers
         DEFAULT_REVIEWER_WEIGHT = Gitlab::Dangerfiles::Weightage::CAPACITY_MULTIPLIER * Gitlab::Dangerfiles::Weightage::BASE_REVIEWER_WEIGHT
         TRAINTAINER_WEIGHT = 2
 
-        def initialize(reviewers, traintainers)
+        # @param reviewers [Array<Gitlab::Dangerfiles::Teammate>]
+        # @param traintainers [Array<Gitlab::Dangerfiles::Teammate>]
+        # @param codeowners_usernames [Set<String>] usernames of CODEOWNERS-eligible approvers for the MR
+        # @param codeowners_boost [Integer] extra weight entries for CODEOWNERS-eligible reviewers
+        def initialize(reviewers, traintainers, codeowners_usernames: Set.new, codeowners_boost: 0)
           @reviewers = reviewers
           @traintainers = traintainers
+          @codeowners_usernames = codeowners_usernames
+          @codeowners_boost = codeowners_boost
         end
 
         def execute
-          # TODO: take CODEOWNERS into account?
-          # https://gitlab.com/gitlab-org/gitlab/issues/26723
-
           remove_traintainers_from_reviewers!
           remove_maintainer_only_from_reviewers!
 
@@ -40,7 +47,7 @@ module Gitlab
 
         private
 
-        attr_reader :reviewers, :traintainers
+        attr_reader :reviewers, :traintainers, :codeowners_usernames, :codeowners_boost
 
         def remove_traintainers_from_reviewers!
           # Sometimes folks will add themselves as traintainers and not remove themselves as reviewers.
@@ -75,6 +82,8 @@ module Gitlab
           else
             reviewers.fill(reviewer, reviewers.size, DEFAULT_REVIEWER_WEIGHT)
           end
+
+          reviewers.fill(reviewer, reviewers.size, codeowners_boost) if codeowners_boost.positive? && codeowners_usernames.include?(reviewer.username)
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 4.11.1
+  version: 4.13.0
 platform: ruby
 authors:
 - GitLab
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-21 00:00:00.000000000 Z
+date: 2026-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -241,14 +241,12 @@ files:
 - lefthook.yml
 - lib/danger/plugins/changelog.rb
 - lib/danger/plugins/commit_messages.rb
-- lib/danger/plugins/duo_code.rb
 - lib/danger/plugins/internal/helper.rb
 - lib/danger/plugins/roulette.rb
 - lib/danger/rules/changelog/Dangerfile
 - lib/danger/rules/changes_size/Dangerfile
 - lib/danger/rules/commit_messages/Dangerfile
 - lib/danger/rules/commits_counter/Dangerfile
-- lib/danger/rules/duo_code_review/Dangerfile
 - lib/danger/rules/metadata/Dangerfile
 - lib/danger/rules/simple_roulette/Dangerfile
 - lib/danger/rules/type_label/Dangerfile

data/lib/danger/plugins/duo_code.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-module Danger
-  # Contains method to check if Duo code suggestion added to MR as a reviewer.
-  class DuoCode < Danger::Plugin
-    def suggestion_added?
-      return false unless helper.ci?
-
-      helper.mr_reviewers.include?("GitLabDuo")
-    end
-  end
-end

data/lib/danger/rules/duo_code_review/Dangerfile

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-return if duo_code.suggestion_added?
-
-case helper.config.duo_code_review
-when :optional
-  markdown("We advise getting a review from GitLab Duo Code. You can assign `@GitLabDuo` as a reviewer to this merge request.")
-when :mandatory
-  failure("A review from GitLab Duo Code is mandatory. Please assign `@GitLabDuo` as a reviewer to this merge request.")
-end