Skip to content

Commit

Permalink
[Ruby] Refactor/update minimum ruby (#370)
Browse files Browse the repository at this point in the history
* Update minimum ruby to 3.2

* Update CI to run on ruby 3.2-3,4
  • Loading branch information
luke-hill authored Feb 8, 2025
1 parent c2c3ef7 commit 40b4cec
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/release-rubygem.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v4
- uses: ruby/setup-ruby@v1
with:
ruby-version: '3.3'
ruby-version: '3.4'
bundler-cache: true
- uses: cucumber/action-publish-rubygem@v1.0.0
with:
Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/test-ruby.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,12 @@ jobs:
matrix:
os:
- ubuntu-latest
ruby: ['3.0', '3.1', '3.2', '3.3']
ruby: ['3.2', '3.3', '3.4']
include:
- os: macos-latest
ruby: '3.3'
ruby: '3.4'
- os: windows-latest
ruby: '3.4'

steps:
- uses: actions/checkout@v4
Expand Down
2 changes: 1 addition & 1 deletion ruby/.rubocop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ inherit_mode:
# - rubocop-rspec

AllCops:
TargetRubyVersion: 3.0
TargetRubyVersion: 3.2
NewCops: enable

# Disabled on our repo's to enable polyglot-release
Expand Down
2 changes: 1 addition & 1 deletion ruby/cucumber-gherkin.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Gem::Specification.new do |s|
s.homepage = 'https://github.com/cucumber/gherkin'
s.platform = Gem::Platform::RUBY
s.license = 'MIT'
s.required_ruby_version = '>= 3.0'
s.required_ruby_version = '>= 3.2'
s.required_rubygems_version = '>= 3.2.8'

s.metadata = {
Expand Down
1 change: 0 additions & 1 deletion ruby/spec/spec_helper.rb
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# frozen_string_literal: true

require 'rspec'

require 'gherkin'

0 comments on commit 40b4cec

Please sign in to comment.