Skip to content

Allow Ruby 4.0.0; add test suite, CI, and CHANGELOG#1

Merged
kleintom merged 4 commits into
mainfrom
allow-ruby-4
Jun 18, 2026
Merged

Allow Ruby 4.0.0; add test suite, CI, and CHANGELOG#1
kleintom merged 4 commits into
mainfrom
allow-ruby-4

Conversation

@gdower

@gdower gdower commented Jun 17, 2026

Copy link
Copy Markdown
Member
  • Relaxed required_ruby_version to >= 2.5.0, < 5.0
  • Bumped faraday-follow_redirects upper bound to allow 0.5+ (which lifts the Ruby < 4 cap)
  • Added test/ suite (test-unit + VCR + webmock) in the style of colrapi, covering version, ontologies, and search
  • Added Rake::TestTask and test default task
  • Added GitHub Actions workflow with a Ruby 3.3.4 / 4.0.0 matrix
  • Added CHANGELOG.md
  • Bumped version to 0.1.1

gdower and others added 4 commits June 17, 2026 11:44
- Relaxed `required_ruby_version` to `>= 2.5.0, < 5.0`
- Bumped `faraday-follow_redirects` upper bound to allow 0.5+ (which
  lifts the Ruby < 4 cap)
- Added test/ suite (test-unit + VCR + webmock) in the style of colrapi,
  covering version, ontologies, and search
- Added Rake::TestTask and test default task
- Added GitHub Actions workflow with a Ruby 3.3.4 / 4.0.0 matrix
- Added CHANGELOG.md
- Bumped version to 0.1.1

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
CI re-records on every run against the live OLS API.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Needed in order for ./bin/console to work
Correct me if wrong, but I think usually Gemfile.lock is not included in gems because it makes it harder for gem users to manage dependencies.
runs-on: ubuntu-latest
strategy:
matrix:
ruby-version: ['3.3.4', '4.0.0']

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Probably should be ['3.3', '3.4', '4.0'] so it used latest patch version of each minor?

@kleintom kleintom merged commit 57fc336 into main Jun 18, 2026
5 of 6 checks passed
@kleintom

Copy link
Copy Markdown
Collaborator

Thanks @gdower, merging this and then I'll make @LocoDelAssembly's suggested change and release.

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