gitlab-dangerfiles 4.1.0 → 4.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c7a2df8349c1deecf91c154f6d8a8fa3e011217b00edb9078a12ae29e04891e9
-  data.tar.gz: 1df827117d0e4012436e5b77e19daa1ee36855e5085581baa29a8e36f4fbd04f
+  metadata.gz: 5721484d3cb437db3c18e94329677c1c170971940a3aa9894f9405d22fc38a7e
+  data.tar.gz: 6df403c9b28c59327d053c75aec9ecb2862cedc201a7ab7191d79dbb8701529b
 SHA512:
-  metadata.gz: f0f4c1dd6bc142deaa101e4b47b362cfab5955b0d4fa4fd13f495bd9f7ce10f9452c1ed050e452aeb808440ba520b18d67533e52e93efd748e8bfd0a8d9c00c1
-  data.tar.gz: 17a595ead4bffa6416632543196f0b4b1f1141ab69f22144a9996f1db0dc76be6a39f3781794fd418333752e6e094042533841b9ea760e6f721c525f868ccf5f
+  metadata.gz: bd1f28e65a9195640bda8ba9277b4c8db884b060009c39589022ebd35e41b6d44434a2eca5d502dc49c294f31ce0fc8542f9cbcfbc25314f0688b8b4286ae645
+  data.tar.gz: 8c3acc9ca82bbcae28741f81c6fa632bee91610bb1ebda8257878ffb704e5a1bf37f06b14d7eaaf03dc489643f4f42357ef27598bc898a54a3cfd1a77a469bc0

data/.gitlab-ci.yml

@@ -46,15 +46,6 @@ test:rubocop:
     matrix:
       - RUBY_VERSION: ['3.0', '3.1', '3.2']
 
-test:rufo:
-  extends: .default
-  stage: test
-  script:
-    - bundle exec rufo --check .
-  parallel:
-    matrix:
-      - RUBY_VERSION: ['3.0', '3.1', '3.2']
-
 include:
   - template: Security/Dependency-Scanning.gitlab-ci.yml
   - template: Security/SAST.gitlab-ci.yml

data/.rubocop.yml

@@ -25,6 +25,9 @@ RSpec/MultipleMemoizedHelpers:
 RSpec/FilePath:
   Enabled: false
 
+RSpec/SpecFilePathFormat:
+  Enabled: false
+
 Style/HashSyntax:
   EnforcedStyle: ruby19_no_mixed_keys
   # Introduced in Ruby 3.1. Disable for now.

data/.rubocop_todo.yml

@@ -519,3 +519,8 @@ Style/TrailingCommaInArrayLiteral:
     - 'spec/gitlab/dangerfiles/title_linting_spec.rb'
     - 'spec/gitlab/dangerfiles/weightage/maintainers_spec.rb'
     - 'spec/gitlab/dangerfiles/weightage/reviewers_spec.rb'
+
+Style/ArrayIntersect:
+  Exclude:
+    - 'lib/gitlab/dangerfiles/category.rb'
+    - 'lib/danger/plugins/changelog.rb'

data/Gemfile

@@ -2,19 +2,3 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in gitlab-dangerfiles.gemspec
 gemspec
-
-gem "rake", "~> 12.0"
-gem "guard-rspec"
-gem "yard"
-
-group :development do
-  gem "lefthook", require: false
-end
-
-group :development, :test do
-  gem "gitlab-styles", "~> 9.0.0", require: false
-end
-
-group :test do
-  gem "pry-byebug", "~> 3.8", require: false
-end

data/gitlab-dangerfiles.gemspec

@@ -27,14 +27,21 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rake", ">= 0"
+  spec.add_dependency "rake", "~> 10.2"
   spec.add_dependency "danger-gitlab", ">= 8.0.0"
   spec.add_dependency "danger", ">= 9.3.0"
 
-  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency "climate_control"
+  spec.add_development_dependency "gitlab-styles", "~> 10.0"
+  spec.add_development_dependency "guard-rspec", "~> 4.7.3"
+  spec.add_development_dependency "lefthook", "~> 1.3"
   spec.add_development_dependency "rspec-parameterized"
+  spec.add_development_dependency "rspec", "~> 3.8"
+  # we do not commit the bundle lockfile, so this temporary workaround needs to be
+  # present until 2.21.3 or 2.22.x is released
+  # See https://gitlab.com/gitlab-org/ruby/gems/gitlab-styles/-/issues/63
+  spec.add_development_dependency "rubocop-rails", "!= 2.21.2"
   spec.add_development_dependency "timecop"
   spec.add_development_dependency "webmock"
-  spec.add_development_dependency "climate_control"
-  spec.add_development_dependency "rufo"
+  spec.add_development_dependency "yard"
 end

data/lib/danger/plugins/roulette.rb

@@ -322,9 +322,9 @@ module Danger
       rsp = Net::HTTP.get_response(URI.parse(url))
 
       if rsp.is_a?(Net::HTTPRedirection)
-        if (uri = URI.parse(rsp.header["location"]))
-          uri.query = nil
-        end
+        uri = URI.parse(rsp.header["location"])
+
+        uri.query = nil if uri
 
         warnings << "Redirection detected: #{uri}."
         return nil

data/lib/danger/rules/simple_roulette/Dangerfile

@@ -95,9 +95,9 @@ if changes.any?
 
   markdown(MESSAGE)
 
-  if (warnings = warning_list(roulette))
-    markdown(format(WARNING_MESSAGE, warnings: warnings))
-  end
+  warnings = warning_list(roulette)
+
+  markdown(format(WARNING_MESSAGE, warnings: warnings)) if warnings
 
   markdown(TABLE_MARKDOWN + table_header + rows.join("\n")) unless rows.empty?
 end

data/lib/danger/rules/z_add_labels/Dangerfile

@@ -6,8 +6,8 @@ return unless helper.ci?
 
 def post_labels(labels)
   gitlab.api.update_merge_request(helper.mr_target_project_id,
-                                  helper.mr_iid,
-                                  add_labels: labels.join(","))
+    helper.mr_iid,
+    add_labels: labels.join(","))
 rescue Gitlab::Error::Forbidden
   warn("This Merge Request needs to be labelled with #{helper.labels_list(labels)}. Please request a reviewer or maintainer to add them.")
 end

data/lib/gitlab/dangerfiles/teammate.rb

@@ -6,7 +6,7 @@ module Gitlab
   module Dangerfiles
     class Teammate
       attr_reader :options, :username, :name, :role, :specialty, :projects, :available, :hungry, :reduced_capacity, :tz_offset_hours,
-                  :only_maintainer_reviews
+        :only_maintainer_reviews
 
       # The options data are produced by https://gitlab.com/gitlab-org/gitlab-roulette/-/blob/main/lib/team_member.rb
       def initialize(options = {})
@@ -14,7 +14,7 @@ module Gitlab
         @username = options["username"]
         @name = options["name"]
         @markdown_name = options["markdown_name"] ||
-                         default_markdown_name(*options.values_at("username", "name"))
+                         default_markdown_name(options["username"])
         @role = options["role"]
         @specialty = options["specialty"]
         @projects = process_projects(options["projects"])
@@ -89,8 +89,8 @@ module Gitlab
 
       private
 
-      def default_markdown_name(username, name)
-        "[#{name}](https://gitlab.com/#{username}) (`@#{username}`)"
+      def default_markdown_name(username)
+        "[`#{username}`](https://gitlab.com/#{username})"
       end
 
       def process_projects(projects)

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "4.1.0"
+    VERSION = "4.2.1"
   end
 end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 4.1.0
+  version: 4.2.1
 platform: ruby
 authors:
 - GitLab
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-09-29 00:00:00.000000000 Z
+date: 2023-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.2'
 - !ruby/object:Gem::Dependency
   name: danger-gitlab
   requirement: !ruby/object:Gem::Requirement
@@ -53,19 +53,61 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 9.3.0
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: climate_control
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: gitlab-styles
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 4.7.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 4.7.3
+- !ruby/object:Gem::Dependency
+  name: lefthook
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rspec-parameterized
   requirement: !ruby/object:Gem::Requirement
@@ -81,21 +123,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: timecop
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.8'
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "!="
+      - !ruby/object:Gem::Version
+        version: 2.21.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "!="
+      - !ruby/object:Gem::Version
+        version: 2.21.2
+- !ruby/object:Gem::Dependency
+  name: timecop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +165,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: climate_control
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +179,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rufo
+  name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="