gitlab-dangerfiles 2.2.1 → 2.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 10b4151926ec3f1b573d1891138cca88e74e349484914f806307639e31aa8f5c
-  data.tar.gz: 3cfcb8533e082e7b157f6a99ea9e84aa3d357cd4035270bcfd717e2212182611
+  metadata.gz: b0c3470d9595a2aa541734ad57d7a47c189c89e6766f59b48baf8928bbd12a81
+  data.tar.gz: c97021338170135f2819563af31d9c8580e44e0b74c2a689c0f764d391ca8718
 SHA512:
-  metadata.gz: 50fe7e8cd2e7ca176639e416d6406f3ab327b95f61e7f07c1d329561cb172bbb73ca8cffb775d14dd4c4b3d375bdba5fd044ee9cc344298f8f27b3195d0765f1
-  data.tar.gz: debd9b7c5a598ee1bb2274900cd61a00b00a5ef96ad3af58b3445e30ba078ed49a5efa86976c11542e0b2652b4fb792604c16654c3f616315a37b63352c145cf
+  metadata.gz: f27ee9ec4cab2d09e2a4c796868f93c4b9e649fc9bcf16fdd3935c956cee3f4ab756e55a14f66ce82fa810354ac1dad087d09b7e6813fb4797a78372fd46e91f
+  data.tar.gz: e8b704505f0f93ab263207d15c8e0a5a7492f55bb18598c40f1fdec93dc6c79eb6ff1e7280642d71d950365b005d3082c43448c8fab702115e2acfc6d19b0a7b

data/README.md

@@ -7,7 +7,7 @@ The goal of this gem is to centralize Danger plugins and rules that to be used b
 Add this line to your application's Gemfile:
 
 ```ruby
-gem 'gitlab-dangerfiles'
+gem 'gitlab-dangerfiles', require: false
 ```
 
 And then execute:
@@ -29,6 +29,8 @@ $ gem install gitlab-dangerfiles
 In your project's `Dangerfile`, add the following two line to import the plugins and rules from this gem:
 
 ```ruby
+require 'gitlab-dangerfiles'
+
 # Get an instance of Gitlab::Dangerfiles
 gitlab_dangerfiles = Gitlab::Dangerfiles::Engine.new(self)
 

data/lib/danger/plugins/helper.rb

@@ -17,7 +17,9 @@ module Danger
       none: "",
       qa: "~QA",
       test: "~test ~Quality for `spec/features/*`",
+      # Deprecated as of 2.3.0 in favor of tooling
       engineering_productivity: '~"Engineering Productivity" for CI, Danger',
+      tooling: '~"type::tooling" for CI, Danger',
       ci_template: '~"ci::templates"',
       product_intelligence: '~"product intelligence"',
     }.freeze
@@ -211,7 +213,7 @@ module Danger
     #                   +filename_regex+ is the regex pattern to match file names. +changes_regex+ is the regex pattern to
     #                   match changed lines in files that match +filename_regex+
     #
-    # @return [Array<Symbol>] the categories a file is in, e.g., +[:frontend]+, +[:backend]+, or +%i[frontend engineering_productivity]+
+    # @return [Array<Symbol>] the categories a file is in, e.g., +[:frontend]+, +[:backend]+, or +%i[frontend tooling]+
     #   using filename regex (+filename_regex+) and specific change regex (+changes_regex+) from the given +categories+ hash.
     def categories_for_file(filename, categories)
       _, categories = categories.find do |key, _|

data/lib/danger/plugins/roulette.rb

@@ -58,7 +58,7 @@ module Danger
             # Fetch an already picked backend reviewer, or pick one otherwise
             spin.reviewer = backend_spin&.reviewer || spin_for_category(project, :backend, timezone_experiment: including_timezone).reviewer
           end
-        when :engineering_productivity
+        when :tooling, :engineering_productivity # Deprecated as of 2.3.0 in favor of tooling
           if spin.maintainer.nil?
             # Fetch an already picked backend maintainer, or pick one otherwise
             spin.maintainer = backend_spin&.maintainer || spin_for_category(project, :backend, timezone_experiment: including_timezone).maintainer

data/lib/danger/rules/commit_messages/Dangerfile

@@ -5,7 +5,7 @@ require_relative "../../../gitlab/dangerfiles/merge_request_linter"
 
 COMMIT_MESSAGE_GUIDELINES = "https://docs.gitlab.com/ee/development/contributing/merge_request_workflow.html#commit-messages-guidelines"
 MORE_INFO = "For more information, take a look at our [Commit message guidelines](#{COMMIT_MESSAGE_GUIDELINES})."
-THE_DANGER_JOB_TEXT = "the `%<job_name>` job"
+THE_DANGER_JOB_TEXT = "the `%<job_name>s` job"
 MAX_COMMITS_COUNT = helper.config.max_commits_count
 MAX_COMMITS_COUNT_EXCEEDED_MESSAGE = <<~MSG
 This merge request includes more than %<max_commits_count>d commits. Each commit should meet the following criteria:

data/lib/gitlab/dangerfiles/base_linter.rb

@@ -28,6 +28,10 @@ module Gitlab
         @problems = {}
       end
 
+      def inspect
+        "#{self.class}:#{object_id} @commit=#{@commit} @problems=#{@problems}"
+      end
+
       def failed?
         problems.any?
       end

data/lib/gitlab/dangerfiles/commit_linter.rb

@@ -9,9 +9,9 @@ module Gitlab
       MAX_CHANGED_FILES_IN_COMMIT = 3
       MAX_CHANGED_LINES_IN_COMMIT = 30
       # Issue, MR, Epic
-      SHORT_REFERENCE_REGEX = %r{([\w\-\/]+)?(?<!`)(#|!|&)\d+(?<!`)}.freeze
+      SHORT_REFERENCE_REGEX = %r{(\S*([\w\-\/]+)?(?<!`)(#|!|&)\d+(?<!`))}.freeze
       # Milestone
-      MS_SHORT_REFERENCE_REGEX = %r{([\w\-\/]+)?(?<!`)%"?\d{1,3}\.\d{1,3}"?(?<!`)}.freeze
+      MS_SHORT_REFERENCE_REGEX = %r{(\S*([\w\-\/]+)?(?<!`)%"?\d{1,3}\.\d{1,3}"?(?<!`))}.freeze
       SUGGESTIONS_APPLIED_COMMIT_REGEX = /Apply \d+ suggestion\(s\) to \d+ file\(s\)/.freeze
 
       def self.problems_mapping
@@ -143,8 +143,13 @@ module Gitlab
       end
 
       def message_contains_short_reference?
-        commit.message.match?(SHORT_REFERENCE_REGEX) ||
-          commit.message.match?(MS_SHORT_REFERENCE_REGEX)
+        match_data = commit.message.match(SHORT_REFERENCE_REGEX) ||
+                     commit.message.match(MS_SHORT_REFERENCE_REGEX)
+
+        return false unless match_data
+
+        # Any URL would include "//". This works for http/https/ftp etc.
+        !match_data[1].include?("//")
       end
 
       def emoji_checker

data/lib/gitlab/dangerfiles/teammate.rb

@@ -95,7 +95,7 @@ module Gitlab
           area = role[/Software Engineer in Test(?:.*?, (\w+))/, 1]
 
           area && labels.any?("devops::#{area.downcase}") if kind == :reviewer
-        when :engineering_productivity
+        when :tooling, :engineering_productivity # Deprecated as of 2.3.0 in favor of tooling
           return false unless role[/Engineering Productivity/]
           return true if kind == :reviewer
           return true if capabilities(project).include?("#{kind} engineering_productivity")

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "2.2.1"
+    VERSION = "2.3.2"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 2.2.1
+  version: 2.3.2
 platform: ruby
 authors:
 - GitLab
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-07-07 00:00:00.000000000 Z
+date: 2021-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-gitlab