gitlab-dangerfiles 2.9.2 → 2.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18ad65769d07e0c6739e489f3c8edf8d008a62fc25c4789476360f073d5bf3b8
-  data.tar.gz: dd5598930037ead15d2f11bea7a0138ea374b143310adcd258da12569dcd0a21
+  metadata.gz: fb3512fc2c750751c44df16f13e6d8d7c47b6a87f77030852b278725503d3994
+  data.tar.gz: b611380c2b6a8f9cd4372e6b63b7b6c13a125373551a706b9e4e11cf552bc270
 SHA512:
-  metadata.gz: e8c68d332a63e4c15b41e17024b5dff040db820c1cfb12929adc88daf8a1624dd1bfe76444a6423134cf301f89faad3b59ba1aa80c859342a80395c472669b72
-  data.tar.gz: d471bed2793b86906d7f8460318c21bf2878d8c2dd4bdf381c925d5ad4d70c0bc36f5b40c63c6c17051cf07866611793baa464eab47756d227066a6f79cc8b2c
+  metadata.gz: ba7680e6aecd1c1231118033e95cbed567a97f638467d2ce0a131ac5def26c0746e19c8117df0629f1c61a3b14706e56bde80945fe3ff3617f0d0a5b87d5e14f
+  data.tar.gz: 5a1df455e26146be92687f2e416f095f6ed9f7176576d13afce1d4a2d38ad0e928b7051b091323a0dc85e5f6a7b74e4fabf09b1ec88b447d089dea9ab9260844

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2020-2021 GitLab
+Copyright (c) 2020-present GitLab
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -131,7 +131,17 @@ project. To use it in your project, perform the following steps:
 
 #### `type_label`
 
-This rule ensures the merge request has a proper [type label](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification) set..
+This rule ensures the merge request has a proper [type label](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification) set.
+
+If the `changelog` plugin is available, it also tries to infer a type label from the `Changelog` trailer of the MR.
+
+#### `z_add_labels`
+
+This rule adds labels set from other rules (via `helper.labels_to_add`), with a single API request.
+
+#### `z_retry_link`
+
+This rule adds a retry link to the job where Danger ran at the end of the Danger message, only if there's any other message to post.
 
 ### CI configuration
 

data/lib/danger/plugins/changelog.rb

@@ -40,14 +40,17 @@ module Danger
         This merge request requires a changelog entry because it [%<reason>s](https://docs.gitlab.com/ee/development/changelog.html#what-warrants-a-changelog-entry).
       MSG
     }.freeze
-
-    def self.categories
-      @categories ||= YAML
-        .load_file("#{ENV["CI_PROJECT_DIR"]}/.gitlab/changelog_config.yml")
-        .fetch("categories")
-        .keys
-        .freeze rescue []
-    end
+    CHANGELOG_CONFIG_FILE = "#{ENV["CI_PROJECT_DIR"]}/.gitlab/changelog_config.yml"
+    DEFAULT_CHANGELOG_CATEGORIES = %w[
+      added
+      fixed
+      changed
+      deprecated
+      removed
+      security
+      performance
+      other
+    ].freeze
 
     class ChangelogCheckResult
       attr_reader :errors, :warnings, :markdowns, :messages
@@ -90,6 +93,24 @@ module Danger
       end
     end
 
+    class CommitWrapper
+      extend Forwardable
+
+      attr_reader :category, :trailer_key
+
+      def initialize(commit, trailer_key, category)
+        @commit = commit
+        @trailer_key = trailer_key
+        @category = category
+      end
+
+      delegate %i[message sha] => :@commit
+    end
+
+    def categories
+      valid_changelog_commits.map(&:category)
+    end
+
     # rubocop:disable Style/SignalException
     def check!
       if git.modified_files.include?("CHANGELOG.md")
@@ -126,17 +147,13 @@ module Danger
     end
 
     def check_changelog_trailer(commit)
-      trailer = commit.message.match(CHANGELOG_TRAILER_REGEX)
-      name = trailer[:name]
-      category = trailer[:category]
-
-      unless name == "Changelog"
-        return ChangelogCheckResult.error("The changelog trailer for commit #{commit.sha} must be `Changelog` (starting with a capital C), not `#{name}`")
+      unless commit.trailer_key == "Changelog"
+        return ChangelogCheckResult.error("The changelog trailer for commit #{commit.sha} must be `Changelog` (starting with a capital C), not `#{commit.trailer_key}`")
       end
 
-      return ChangelogCheckResult.empty if self.class.categories.include?(category)
+      return ChangelogCheckResult.empty if valid_categories.include?(commit.category)
 
-      ChangelogCheckResult.error("Commit #{commit.sha} uses an invalid changelog category: #{category}")
+      ChangelogCheckResult.error("Commit #{commit.sha} uses an invalid changelog category: #{commit.category}")
     end
 
     def check_changelog_path
@@ -180,16 +197,16 @@ module Danger
     end
 
     def changelog_commits
-      git.commits.select do |commit|
-        commit.message.match?(CHANGELOG_TRAILER_REGEX)
+      git.commits.each_with_object([]) do |commit, memo|
+        trailer = commit.message.match(CHANGELOG_TRAILER_REGEX)
+
+        memo << CommitWrapper.new(commit, trailer[:name], trailer[:category]) if trailer
       end
     end
 
     def valid_changelog_commits
       changelog_commits.select do |commit|
-        trailer = commit.message.match(CHANGELOG_TRAILER_REGEX)
-
-        self.class.categories.include?(trailer[:category])
+        valid_categories.include?(commit.message.match(CHANGELOG_TRAILER_REGEX)[:category])
       end
     end
 
@@ -219,6 +236,28 @@ module Danger
 
     private
 
+    def valid_categories
+      return @categories if defined?(@categories)
+
+      @categories = if File.exist?(CHANGELOG_CONFIG_FILE)
+          begin
+            YAML
+              .load_file(CHANGELOG_CONFIG_FILE)
+              .fetch("categories")
+              .keys
+              .freeze
+          rescue Psych::SyntaxError, Psych::DisallowedClass => ex
+            puts "#{CHANGELOG_CONFIG_FILE} doesn't seem to be a valid YAML file:\n#{ex.message}\nFallbacking to the default categories: #{DEFAULT_CHANGELOG_CATEGORIES}"
+            DEFAULT_CHANGELOG_CATEGORIES
+          rescue => ex
+            puts "Received an unexpected failure while trying to fetch categories at #{CHANGELOG_CONFIG_FILE}:\n#{ex.message}\nFallbacking to the default categories: #{DEFAULT_CHANGELOG_CATEGORIES}"
+            DEFAULT_CHANGELOG_CATEGORIES
+          end
+        else
+          DEFAULT_CHANGELOG_CATEGORIES
+        end
+    end
+
     def read_file(path)
       File.read(path)
     end

data/lib/danger/plugins/internal/helper.rb

@@ -120,7 +120,7 @@ module Danger
     #   +modified_files+ might contain paths that already have been renamed,
     #   so we need to remove them from the list.
     def all_changed_files
-      changes.files - changes.renamed_before.files
+      changes.files - changes.deleted.files - changes.renamed_before.files
     end
 
     # @param filename [String] A file name for which we want the diff.

data/lib/danger/rules/type_label/Dangerfile

@@ -1,5 +1,12 @@
 # frozen_string_literal: true
 
+require_relative "../../../gitlab/dangerfiles/type_label_guesser"
+
+if respond_to?(:changelog) && !helper.has_scoped_label_with_scope?("type")
+  type_label_guesser = Gitlab::Dangerfiles::TypeLabelGuesser.new
+  helper.labels_to_add.merge(type_label_guesser.labels_from_changelog_categories(changelog.categories))
+end
+
 unless helper.has_scoped_label_with_scope?("type")
   warn "Please add a [merge request type](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification) to this merge request."
 end

data/lib/danger/rules/z_add_labels/Dangerfile

@@ -1,4 +1,4 @@
-# This rule is the last one since we want to add all the labels to add at once,
+# This rule is the (almost) last one since we want to add all the labels to add at once,
 # so we let other rules adding to `helper.labels_to_add` before actually adding them via the API here.
 
 # Don't try to post anything locally.
@@ -13,9 +13,3 @@ rescue Gitlab::Error::Forbidden
 end
 
 post_labels(helper.labels_to_add) if helper.labels_to_add.any?
-
-anything_to_post = status_report.values.any?(&:any?)
-
-if anything_to_post
-  markdown("**If needed, you can retry the [`danger-review` job](#{ENV["CI_JOB_URL"]}) that generated this comment.**")
-end

data/lib/danger/rules/z_retry_link/Dangerfile

@@ -0,0 +1,10 @@
+# This rule is the last one since we only want to post the "retry" message if there are anything to be posted.
+
+# There's no job to retry locally.
+return unless helper.ci?
+
+anything_to_post = status_report.values.any?(&:any?)
+
+if anything_to_post
+  markdown("**If needed, you can retry the [🔁 `danger-review` job](#{ENV['CI_JOB_URL']}) that generated this comment.**")
+end

data/lib/gitlab/dangerfiles/commit_linter.rb

@@ -115,11 +115,15 @@ module Gitlab
       end
 
       def files_changed
-        commit.diff_parent.stats[:total][:files]
+        # In some cases, the commits cannot be found so it's better to just move one
+        # See https://gitlab.com/gitlab-org/gitlab/-/issues/227814.
+        commit.diff_parent.size rescue 0
       end
 
       def lines_changed
-        commit.diff_parent.stats[:total][:lines]
+        # In some cases, the commits cannot be found so it's better to just move one
+        # See https://gitlab.com/gitlab-org/gitlab/-/issues/227814.
+        commit.diff_parent.lines rescue 0
       end
 
       def many_changes?

data/lib/gitlab/dangerfiles/type_label_guesser.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Dangerfiles
+    class TypeLabelGuesser
+      CHANGELOG_CATEGORY_TO_TYPE_LABEL = {
+        fixed: %w[type::bug],
+        security: %w[type::bug bug::vulnerability],
+        performance: %w[type::bug bug::performance],
+        added: %w[type::feature feature::addition],
+        deprecated: %w[type::feature feature::removal],
+        removed: %w[type::feature feature::removal],
+      }.freeze
+
+      def labels_from_changelog_categories(categories)
+        categories = categories.map(&:to_sym) & CHANGELOG_CATEGORY_TO_TYPE_LABEL.keys
+        return [] unless categories.one?
+
+        CHANGELOG_CATEGORY_TO_TYPE_LABEL.fetch(categories.first.to_sym, [])
+      end
+    end
+  end
+end

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "2.9.2"
+    VERSION = "2.10.1"
   end
 end

data/lib/gitlab/dangerfiles.rb

@@ -17,6 +17,7 @@ module Gitlab
       simple_roulette
       type_label
       z_add_labels
+      z_retry_link
     ].freeze
 
     # Utility method to construct a [Gitlab::Dangerfiles::Engine] instance,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 2.9.2
+  version: 2.10.1
 platform: ruby
 authors:
 - GitLab
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-02-23 00:00:00.000000000 Z
+date: 2022-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-gitlab
@@ -158,6 +158,7 @@ files:
 - lib/danger/rules/simple_roulette/Dangerfile
 - lib/danger/rules/type_label/Dangerfile
 - lib/danger/rules/z_add_labels/Dangerfile
+- lib/danger/rules/z_retry_link/Dangerfile
 - lib/gitlab-dangerfiles.rb
 - lib/gitlab/Dangerfile
 - lib/gitlab/dangerfiles.rb
@@ -170,6 +171,7 @@ files:
 - lib/gitlab/dangerfiles/spec_helper.rb
 - lib/gitlab/dangerfiles/teammate.rb
 - lib/gitlab/dangerfiles/title_linting.rb
+- lib/gitlab/dangerfiles/type_label_guesser.rb
 - lib/gitlab/dangerfiles/version.rb
 - lib/gitlab/dangerfiles/weightage.rb
 - lib/gitlab/dangerfiles/weightage/maintainers.rb