From 891a3967e54b84a0f57f8343e714513ab975a4d8 Mon Sep 17 00:00:00 2001 From: Renato Arruda Date: Tue, 15 Nov 2022 23:10:40 +0100 Subject: [PATCH] fix gemspec - appease rubocop - more fixes apparently i don't run all tests in all platforms locally --- Steepfile | 4 ++++ unleash-client.gemspec | 9 +++++++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/Steepfile b/Steepfile index 71a612e3..88ceffb9 100644 --- a/Steepfile +++ b/Steepfile @@ -3,10 +3,14 @@ D = Steep::Diagnostic target :lib do signature "*.rbs" + # NOTE: All client-exposed methods/types should now have type signatures / rbs support check "lib/unleash/client.rb" check "lib/unleash/context.rb" + check "lib/unleash/variant.rb" check "lib/unleash/scheduled_executor.rb" + # TODO: add signatures to the rest + # Mostly internal SDK files. ignore "lib/unleash/bootstrap" ignore "lib/unleash/strategy/*.rb" ignore "lib/unleash/util" diff --git a/unleash-client.gemspec b/unleash-client.gemspec index c05b6811..3b53a9b5 100644 --- a/unleash-client.gemspec +++ b/unleash-client.gemspec @@ -40,6 +40,11 @@ Gem::Specification.new do |spec| spec.add_development_dependency "simplecov", "~> 0.21.2" spec.add_development_dependency "simplecov-lcov", "~> 0.8.0" - spec.add_development_dependency "rbs", "~> 2.7" - spec.add_development_dependency "steep", "~> 1.2" + # NOTE: only require rbs/steep in supported ruby versions. In EOL ruby/jruby, just ignore. + # rubocop:disable Gemspec/RubyVersionGlobalsUsage + if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.6') && RUBY_ENGINE != 'jruby' + spec.add_development_dependency "rbs", "~> 2.8" + spec.add_development_dependency "steep", "~> 1.3" + end + # rubocop:enable Gemspec/RubyVersionGlobalsUsage end