Skip to content

Conversation

@gs-deliverists-io
Copy link
Contributor

Summary

  • Add Ruby 4.0 support to the gem by updating the required_ruby_version constraint
  • Update GitHub Actions CI workflow to test against Ruby 4.0 with multiple Node.js versions
  • Fix spec helper to handle nil puppeteer version gracefully
  • Update CHANGELOG with Ruby 4.0 support

Changes

  • Updated grover.gemspec to allow Ruby versions up to 4.1.0
  • Updated .github/workflows/test.yml to include Ruby 4.0 in the test matrix with Node.js 18, 20, 22, and 24
  • Fixed spec/spec_helper.rb to check for nil before calling empty? on puppeteer version
  • Updated CHANGELOG.md to document Ruby 4.0 support

Testing

All existing tests pass with Ruby 4.0.0. The gem is fully compatible with Ruby 4.0.

puppeteer-version: '24.10.2'
- ruby-version: '3.4'
node-version: '24'
puppeteer-version: '24.10.2'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can also get rid of these

grover.gemspec Outdated
spec.homepage = 'https://github.com/Studiosity/grover'
spec.license = 'MIT'
spec.required_ruby_version = ['>= 3.0.0', '< 3.5.0']
spec.required_ruby_version = ['>= 3.0.0', '< 5.0.0']
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

limiting to < 4.1 is likely safer.. we don't really know what the future of the language will be. In all likelihood it won't be a problem, but that isn't how the history of the language has evolved 😄

@abrom abrom mentioned this pull request Jan 14, 2026
@abrom abrom merged commit 8fc8c85 into Studiosity:main Jan 14, 2026
11 checks passed
@abrom
Copy link
Contributor

abrom commented Jan 14, 2026

Released in v1.2.5

@gs-deliverists-io
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants