summaryrefslogtreecommitdiff
path: root/spec/deprecation_toolkit_env.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/deprecation_toolkit_env.rb')
-rw-r--r--spec/deprecation_toolkit_env.rb10
1 files changed, 6 insertions, 4 deletions
diff --git a/spec/deprecation_toolkit_env.rb b/spec/deprecation_toolkit_env.rb
index 8e06dcb8c75..b95a8c599bf 100644
--- a/spec/deprecation_toolkit_env.rb
+++ b/spec/deprecation_toolkit_env.rb
@@ -47,16 +47,18 @@ module DeprecationToolkitEnv
end
# Taken from https://github.com/jeremyevans/ruby-warning/blob/1.1.0/lib/warning.rb#L18
+ # Note: When a spec fails due to this warning, please update the spec to address the deprecation.
def self.kwargs_warning
%r{warning: (?:Using the last argument (?:for `.+' )?as keyword parameters is deprecated; maybe \*\* should be added to the call|Passing the keyword argument (?:for `.+' )?as the last hash parameter is deprecated|Splitting the last argument (?:for `.+' )?into positional and keyword parameters is deprecated|The called method (?:`.+' )?is defined here)\n\z}
end
- # Allow these Gem paths to trigger keyword warnings as we upgrade these gems
- # one by one
+ # Note: No new exceptions should be added here, unless they are in external dependencies.
+ # In this case, we recommend to add a silence together with an issue to patch or update
+ # the dependency causing the problem.
+ # See https://gitlab.com/gitlab-org/gitlab/-/commit/aea37f506bbe036378998916d374966c031bf347#note_647515736
def self.allowed_kwarg_warning_paths
%w[
- asciidoctor-2.0.12/lib/asciidoctor/extensions.rb
- gitlab-labkit-0.20.0/lib/labkit/correlation/grpc/client_interceptor.rb
+ actionpack-6.1.3.2/lib/action_dispatch/routing/route_set.rb
]
end