Skip to content

Add ruby 4.0 to CI - #5805

Open
Bubballoo3 wants to merge 9 commits into
masterfrom
Bubballoo3-patch-2
Open

Bubballoo3 wants to merge 9 commits into
masterfrom
Bubballoo3-patch-2

Conversation

@Bubballoo3

@Bubballoo3 Bubballoo3 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

What does this PR do, and what is the related issue, if applicable?

Adds Ruby 4.0 to the CI matrix, and will fix any issues that arise in testing. I was unable to keep with the pattern of mentioning which OS required which version of ruby, as I am unable to find any that provide > 3.3 by default. If someone knows which OS requires 3.4 or 4.0, I'd be happy to add a comment on it

Fixes #5802

Testing

  • Tests included
  • If a maintainers' assistance is needed for tests, add label test help
  • No test is needed because _____ (documentation fix, dependency update, etc.)

Documentation

If this PR is for a new feature or behavior change, please help us kick start the documentation with a blurb.

Documentation blurb here

Code Authorship & Understanding

  • I can explain what this code does and answer questions about it
  • This PR was generated or assisted by AI tools (Copilot, Claude, agents). If so, I have reviewed and tested the code
    and I take responsibility for its correctness.

Checklist

  • Follows project code style and conventions
  • This is a large pull request and was discussed first in an issue or with the maintainers.

Anything else?

Screenshots, context, or anything reviewers should know.

Comment thread Gemfile Outdated
gem 'beaker-rspec'
end

gem "ostruct", "~> 0.6.3"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we can get rid of the version requirement here - I don't imagine ostruct to really change APIs ever.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sounds good. This was just the default entry from bundle add ostruct

Comment thread apps/dashboard/Gemfile Outdated
Comment on lines +40 to +41
gem 'i18n-tasks', '~> 1.1'
gem 'highline', '~> 3.0'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Given these are development gems, I wonder if we want to specify the version at all?

Comment thread .github/workflows/tests.yml Outdated
Comment thread .github/workflows/tests.yml Outdated
Comment thread .github/workflows/tests.yml Outdated
- "3.1" # Test versions from Debian 12
- "3.2" # Test versions from Amazon Linux 2023 and Ubuntu 24.04
- "3.3" # Test versions from RHEL8 & RHEL9 & RHEL10 && Ubuntu 26.04
- "3.4"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why 3.4?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I added it when we removed 3.0, but I am not aware of any OS that includes it. Kind of surprised that newer OSs would skip it and go straight to 4.0 though.

Bubballoo3 and others added 2 commits September 17, 2026 16:23
Co-authored-by: treydock <tdockendorf@osc.edu>
Co-authored-by: treydock <tdockendorf@osc.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Awaiting Review

Development

Successfully merging this pull request may close these issues.

Add Ruby 4.0 to CI matrix

4 participants