gitlab-dangerfiles 3.4.0 → 3.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1d8182a9895b7c4ce505c6c31271030222ef4e3eefaee2569edccfe1a3ce892a
-  data.tar.gz: 32f4cd7af125f0cf2a96f7bb000eb725f2c58a7a5576349fe3905dd93302c53d
+  metadata.gz: 17495e36a06253d291cf69ea61fd426ba3541738591873cfdef64e2c9132e8ee
+  data.tar.gz: 5e690f454e356ae1dd7e372ae16d9d8b99b32e1450a48560fe359dfcad20eab9
 SHA512:
-  metadata.gz: fdb2baf7afe55ad315269cf4231efa717099dbb931ca5a6f3d7b268d9551b337fa5265627b8e134d739383a4cf72f5bf68588a19010bd4327c04297baaf3a49f
-  data.tar.gz: bdfa7d3753215241101362459e869cf53b1b4f141e96c58f9a31aac02a1dcb4aadae2caf98115704e8814664f0b74c466781aafe8da02d8f0bd00793857ccfb8
+  metadata.gz: f96558f5954dea10cf802d32003045dcdcb6bccaa00c34b4079a654a275ec8dcb980ca84bf8116d87db1e0dc8a5e85a54b07ec05dc2dbcab4b2696e3e1532173
+  data.tar.gz: b112119bfbbff88bb66966bed0e5ee0cd32369b2eae89507e17f9e602a8c0a987bbd423aa1a4ed089479c1d9f6842f523db76e83c887676c9387cbd6da4db3e3

data/Gemfile

@@ -7,6 +7,10 @@ gem "rake", "~> 12.0"
 gem "guard-rspec"
 gem "yard"
 
+group :development do
+  gem "lefthook", require: false
+end
+
 group :test do
   gem "pry-byebug", "~> 3.8", require: false
 end

data/README.md

@@ -215,12 +215,19 @@ Latest documentation can be found at <https://www.rubydoc.info/gems/gitlab-dange
 
 ## Development
 
+### Initial setup
+
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
 
 To install this gem onto your local machine, run `bundle exec rake install`. 
 
 To release a new version, update the version number in `version.rb`, and get the MR merged by a maintainer. This will be then be packaged into a gem and pushed to [rubygems.org](https://rubygems.org) by the CI/CD.
 
+### Activate lefthook locally
+
+```shell
+lefthook install
+```
 ## Contributing
 
 Bug reports and merge requests are welcome at https://gitlab.com/gitlab-org/gitlab-dangerfiles. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://gitlab.com/gitlab-org/gitlab-dangerfiles/blob/master/CODE_OF_CONDUCT.md).

data/lefthook.yml

@@ -0,0 +1,11 @@
+# EXAMPLE USAGE
+# Refer for explanation to following link:
+# https://github.com/evilmartians/lefthook/blob/master/docs/full_guide.md
+#
+
+pre-push:
+  commands:
+    # Runs RUby FOrmatter (rufo)
+    rufo:
+      run: bundle exec rufo --check .
+      glob: '*.rb'

data/lib/danger/plugins/roulette.rb

@@ -162,12 +162,44 @@ module Danger
       end
     end
 
+    # Spin a reviewer for a particular approval rule
+    #
+    # @param [Hash] rule of approval
+    #
+    # @return [Gitlab::Dangerfiles::Teammate]
     def spin_for_approver(rule)
-      approvers = rule["eligible_approvers"].map do |approver|
+      # This will filter out approvers who are not even reviewers who
+      # don't show up in roulette data we're relying on.
+      # That's why `filter_map` is used.
+      approvers = rule["eligible_approvers"].filter_map do |approver|
         find_member(approver["username"])
       end
 
-      spin_for_person(approvers)
+      spin_for_person(approvers) || spin_for_approver_fallback(rule)
+    end
+
+    # It can be possible that we don't have a valid reviewer for approval.
+    # In this case, we sample again without considering:
+    #
+    # * If they're available
+    # * If they're an actual reviewer from roulette data
+    #
+    # We do this because we strictly require an approval from the approvers.
+    #
+    # @param [Hash] rule of approval
+    #
+    # @return [Gitlab::Dangerfiles::Teammate]
+    def spin_for_approver_fallback(rule)
+      fallback_approvers = rule["eligible_approvers"].map do |approver|
+        find_member(approver["username"]) ||
+          Gitlab::Dangerfiles::Teammate.new(approver)
+      end
+
+      # Intentionally not using `spin_for_person` to skip `valid_person?`.
+      # This should strictly return someone so we don't filter anything,
+      # and it's a fallback mechanism which should not happen often that
+      # deserves a complex algorithm.
+      fallback_approvers.sample(random: random)
     end
 
     def spin_for_category(project, category, timezone_experiment: false)

data/lib/gitlab/dangerfiles/teammate.rb

@@ -10,7 +10,8 @@ module Gitlab
         @options = options
         @username = options["username"]
         @name = options["name"]
-        @markdown_name = options["markdown_name"]
+        @markdown_name = options["markdown_name"] ||
+                         default_markdown_name(*options.values_at("username", "name"))
         @role = options["role"]
         @projects = process_projects(options["projects"])
         @available = options["available"]
@@ -66,7 +67,7 @@ module Gitlab
       end
 
       def markdown_name(author: nil)
-        "#{@markdown_name} (#{utc_offset_text(author)})"
+        "#{@markdown_name}#{utc_offset_text(author)}"
       end
 
       def local_hour
@@ -83,6 +84,10 @@ module Gitlab
 
       private
 
+      def default_markdown_name(username, name)
+        "[#{name}](https://gitlab.com/#{username}) (`@#{username}`)"
+      end
+
       def process_projects(projects)
         return nil unless projects
 
@@ -92,15 +97,19 @@ module Gitlab
       end
 
       def utc_offset_text(author = nil)
+        return unless tz_offset_hours
+
         offset_text = if floored_offset_hours >= 0
             "UTC+#{floored_offset_hours}"
           else
             "UTC#{floored_offset_hours}"
           end
 
-        return offset_text unless author
-
-        "#{offset_text}, #{offset_diff_compared_to_author(author)}"
+        if author
+          " (#{offset_text}, #{offset_diff_compared_to_author(author)})"
+        else
+          " (#{offset_text})"
+        end
       end
 
       def offset_diff_compared_to_author(author)

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "3.4.0"
+    VERSION = "3.4.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 3.4.0
+  version: 3.4.1
 platform: ruby
 authors:
 - GitLab
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-06-16 00:00:00.000000000 Z
+date: 2022-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -163,6 +163,7 @@ files:
 - fixtures/emojis/aliases.json
 - fixtures/emojis/digests.json
 - gitlab-dangerfiles.gemspec
+- lefthook.yml
 - lib/danger/plugins/changelog.rb
 - lib/danger/plugins/internal/helper.rb
 - lib/danger/plugins/roulette.rb