gitlab-dangerfiles 0.8.1 → 2.0.0

data/lib/gitlab/dangerfiles/teammate.rb

@@ -101,6 +101,8 @@ module Gitlab
           return true if capabilities(project).include?("#{kind} engineering_productivity")
 
           capabilities(project).include?("#{kind} backend")
+        when nil
+          capabilities(project).include?("#{kind}")
         else
           capabilities(project).include?("#{kind} #{category}")
         end

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "0.8.1"
+    VERSION = "2.0.0"
   end
 end

data/lib/gitlab/dangerfiles/weightage.rb

@@ -2,6 +2,7 @@
 
 module Gitlab
   module Dangerfiles
+    # @api private
     module Weightage
       CAPACITY_MULTIPLIER = 2 # change this number to change what it means to be a reduced capacity reviewer 1/this number
       BASE_REVIEWER_WEIGHT = 1

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

@@ -5,6 +5,7 @@ require_relative "../weightage"
 module Gitlab
   module Dangerfiles
     module Weightage
+      # @api private
       class Maintainers
         def initialize(maintainers)
           @maintainers = maintainers

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

@@ -18,6 +18,7 @@ module Gitlab
       # | hungry traintainer           |                              8 |
       # +------------------------------+--------------------------------+
       #
+      # @api private
       class Reviewers
         DEFAULT_REVIEWER_WEIGHT = Gitlab::Dangerfiles::Weightage::CAPACITY_MULTIPLIER * Gitlab::Dangerfiles::Weightage::BASE_REVIEWER_WEIGHT
         TRAINTAINER_WEIGHT = 3

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 2.0.0
 platform: ruby
 authors:
-- Rémy Coutable
+- GitLab
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-03-16 00:00:00.000000000 Z
+date: 2021-04-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: danger
+  name: danger-gitlab
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -110,7 +110,7 @@ dependencies:
         version: '0'
 description: This gem provides common Dangerfile and plugins for GitLab projects.
 email:
-- remy@rymai.me
+- gitlab_rubygems@gitlab.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -119,6 +119,7 @@ files:
 - ".gitlab-ci.yml"
 - ".gitlab/merge_request_templates/Release.md"
 - ".rspec"
+- ".yardopts"
 - CODE_OF_CONDUCT.md
 - Gemfile
 - Guardfile
@@ -130,14 +131,16 @@ files:
 - fixtures/emojis/aliases.json
 - fixtures/emojis/digests.json
 - gitlab-dangerfiles.gemspec
-- lib/danger/helper.rb
-- lib/danger/roulette.rb
+- lib/danger/plugins/helper.rb
+- lib/danger/plugins/roulette.rb
+- lib/danger/rules/changes_size/Dangerfile
 - lib/gitlab-dangerfiles.rb
 - lib/gitlab/Dangerfile
 - lib/gitlab/dangerfiles.rb
 - lib/gitlab/dangerfiles/base_linter.rb
 - lib/gitlab/dangerfiles/changes.rb
 - lib/gitlab/dangerfiles/commit_linter.rb
+- lib/gitlab/dangerfiles/config.rb
 - lib/gitlab/dangerfiles/emoji_checker.rb
 - lib/gitlab/dangerfiles/merge_request_linter.rb
 - lib/gitlab/dangerfiles/spec_helper.rb
@@ -154,7 +157,7 @@ metadata:
   allowed_push_host: https://rubygems.org
   homepage_uri: https://gitlab.com/gitlab-org/gitlab-dangerfiles
   source_code_uri: https://gitlab.com/gitlab-org/gitlab-dangerfiles
-  changelog_uri: https://gitlab.com/gitlab-org/gitlab-dangerfiles
+  changelog_uri: https://gitlab.com/gitlab-org/gitlab-dangerfiles/-/releases
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -170,7 +173,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: This gem provides common Dangerfile and plugins for GitLab projects.

data/lib/danger/helper.rb

@@ -1,257 +0,0 @@
-# frozen_string_literal: true
-
-require "net/http"
-require "json"
-require "danger"
-require_relative "../gitlab/dangerfiles/changes"
-require_relative "../gitlab/dangerfiles/teammate"
-require_relative "../gitlab/dangerfiles/title_linting"
-
-module Danger
-  # Common helper functions for our danger scripts.
-  class Helper < Danger::Plugin
-    RELEASE_TOOLS_BOT = "gitlab-release-tools-bot"
-    DRAFT_REGEX = /\A*#{Regexp.union(/(?i)(\[WIP\]\s*|WIP:\s*|WIP$)/, /(?i)(\[draft\]|\(draft\)|draft:|draft\s\-\s|draft$)/)}+\s*/i.freeze
-    CATEGORY_LABELS = {
-      docs: "~documentation", # Docs are reviewed along DevOps stages, so don't need roulette for now.
-      none: "",
-      qa: "~QA",
-      test: "~test ~Quality for `spec/features/*`",
-      engineering_productivity: '~"Engineering Productivity" for CI, Danger',
-      ci_template: '~"ci::templates"',
-    }.freeze
-
-    HTTPError = Class.new(StandardError)
-
-    def gitlab_helper
-      # Unfortunately the following does not work:
-      # - respond_to?(:gitlab)
-      # - respond_to?(:gitlab, true)
-      gitlab
-    rescue NoMethodError
-      nil
-    end
-
-    def html_link(str)
-      ci? ? gitlab_helper.html_link(str) : str
-    end
-
-    def ci?
-      !gitlab_helper.nil?
-    end
-
-    # @param [String] url
-    def http_get_json(url)
-      rsp = Net::HTTP.get_response(URI.parse(url))
-
-      unless rsp.is_a?(Net::HTTPOK)
-        raise HTTPError, "Failed to read #{url}: #{rsp.code} #{rsp.message}"
-      end
-
-      JSON.parse(rsp.body)
-    end
-
-    # Returns a list of all files that have been added, modified or renamed.
-    # `git.modified_files` might contain paths that already have been renamed,
-    # so we need to remove them from the list.
-    #
-    # Considering these changes:
-    #
-    # - A new_file.rb
-    # - D deleted_file.rb
-    # - M modified_file.rb
-    # - R renamed_file_before.rb -> renamed_file_after.rb
-    #
-    # it will return
-    # ```
-    # [ 'new_file.rb', 'modified_file.rb', 'renamed_file_after.rb' ]
-    # ```
-    #
-    # @return [Array<String>]
-    def all_changed_files
-      Set.new
-        .merge(git.added_files.to_a)
-        .merge(git.modified_files.to_a)
-        .merge(git.renamed_files.map { |x| x[:after] })
-        .subtract(git.renamed_files.map { |x| x[:before] })
-        .to_a
-        .sort
-    end
-
-    # Returns a string containing changed lines as git diff
-    #
-    # Considering changing a line in lib/gitlab/usage_data.rb it will return:
-    #
-    # [ "--- a/lib/gitlab/usage_data.rb",
-    #   "+++ b/lib/gitlab/usage_data.rb",
-    #   "+      # Test change",
-    #   "-      # Old change" ]
-    def changed_lines(changed_file)
-      diff = git.diff_for_file(changed_file)
-      return [] unless diff
-
-      diff.patch.split("\n").select { |line| %r{^[+-]}.match?(line) }
-    end
-
-    def release_automation?
-      gitlab_helper&.mr_author == RELEASE_TOOLS_BOT
-    end
-
-    def markdown_list(items)
-      list = items.map { |item| "* `#{item}`" }.join("\n")
-
-      if items.size > 10
-        "\n<details>\n\n#{list}\n\n</details>\n"
-      else
-        list
-      end
-    end
-
-    # @return [Hash<Symbol,Array<String>>]
-    def changes_by_category(categories)
-      all_changed_files.each_with_object(Hash.new { |h, k| h[k] = [] }) do |file, hash|
-        categories_for_file(file, categories).each { |category| hash[category] << file }
-      end
-    end
-
-    # @return [Gitlab::Dangerfiles::Changes]
-    def changes(categories)
-      Gitlab::Dangerfiles::Changes.new([]).tap do |changes|
-        git.added_files.each do |file|
-          categories_for_file(file, categories).each { |category| changes << Gitlab::Dangerfiles::Change.new(file, :added, category) }
-        end
-
-        git.modified_files.each do |file|
-          categories_for_file(file, categories).each { |category| changes << Gitlab::Dangerfiles::Change.new(file, :modified, category) }
-        end
-
-        git.deleted_files.each do |file|
-          categories_for_file(file, categories).each { |category| changes << Gitlab::Dangerfiles::Change.new(file, :deleted, category) }
-        end
-
-        git.renamed_files.map { |x| x[:before] }.each do |file|
-          categories_for_file(file, categories).each { |category| changes << Gitlab::Dangerfiles::Change.new(file, :renamed_before, category) }
-        end
-
-        git.renamed_files.map { |x| x[:after] }.each do |file|
-          categories_for_file(file, categories).each { |category| changes << Gitlab::Dangerfiles::Change.new(file, :renamed_after, category) }
-        end
-      end
-    end
-
-    # Determines the categories a file is in, e.g., `[:frontend]`, `[:backend]`, or  `%i[frontend engineering_productivity]`
-    # using filename regex and specific change regex if given.
-    #
-    # @return Array<Symbol>
-    def categories_for_file(file, categories)
-      _, categories = categories.find do |key, _|
-        filename_regex, changes_regex = Array(key)
-
-        found = filename_regex.match?(file)
-        found &&= changed_lines(file).any? { |changed_line| changes_regex.match?(changed_line) } if changes_regex
-
-        found
-      end
-
-      Array(categories || :unknown)
-    end
-
-    # Returns the GFM for a category label, making its best guess if it's not
-    # a category we know about.
-    #
-    # @return[String]
-    def label_for_category(category)
-      CATEGORY_LABELS.fetch(category, "~#{category}")
-    end
-
-    def new_teammates(usernames)
-      usernames.map { |u| Gitlab::Dangerfiles::Teammate.new("username" => u) }
-    end
-
-    def mr_iid
-      return "" unless gitlab_helper
-
-      gitlab_helper.mr_json["iid"]
-    end
-
-    def mr_title
-      return "" unless gitlab_helper
-
-      gitlab_helper.mr_json["title"]
-    end
-
-    def mr_web_url
-      return "" unless gitlab_helper
-
-      gitlab_helper.mr_json["web_url"]
-    end
-
-    def mr_labels
-      return [] unless gitlab_helper
-
-      gitlab_helper.mr_labels
-    end
-
-    def mr_target_branch
-      return "" unless gitlab_helper
-
-      gitlab_helper.mr_json["target_branch"]
-    end
-
-    def draft_mr?
-      Gitlab::Dangerfiles::TitleLinting.has_draft_flag?(mr_title)
-    end
-
-    def security_mr?
-      mr_web_url.include?("/gitlab-org/security/")
-    end
-
-    def cherry_pick_mr?
-      Gitlab::Dangerfiles::TitleLinting.has_cherry_pick_flag?(mr_title)
-    end
-
-    def run_all_rspec_mr?
-      Gitlab::Dangerfiles::TitleLinting.has_run_all_rspec_flag?(mr_title)
-    end
-
-    def run_as_if_foss_mr?
-      Gitlab::Dangerfiles::TitleLinting.has_run_as_if_foss_flag?(mr_title)
-    end
-
-    def stable_branch?
-      /\A\d+-\d+-stable-ee/i.match?(mr_target_branch)
-    end
-
-    def mr_has_labels?(*labels)
-      labels = labels.flatten.uniq
-
-      (labels & mr_labels) == labels
-    end
-
-    def labels_list(labels, sep: ", ")
-      labels.map { |label| %Q{~"#{label}"} }.join(sep)
-    end
-
-    def prepare_labels_for_mr(labels)
-      return "" unless labels.any?
-
-      "/label #{labels_list(labels, sep: " ")}"
-    end
-
-    def changed_files(regex)
-      all_changed_files.grep(regex)
-    end
-
-    def has_database_scoped_labels?(current_mr_labels)
-      current_mr_labels.any? { |label| label.start_with?("database::") }
-    end
-
-    def has_ci_changes?
-      changed_files(%r{\A(\.gitlab-ci\.yml|\.gitlab/ci/)}).any?
-    end
-
-    def group_label(labels)
-      labels.find { |label| label.start_with?("group::") }
-    end
-  end
-end