Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update locked ruby version to 3.3.1 #2307

Merged
merged 1 commit into from
May 8, 2024
Merged

Update locked ruby version to 3.3.1 #2307

merged 1 commit into from
May 8, 2024

Conversation

onk
Copy link
Collaborator

@onk onk commented May 8, 2024

Follow up for #2283.

BTW, given that our Ruby version is managed locally via .ruby-version and in production with Docker, Ruby version specification in Gemfile might be redundant.
Should we consider removing the Ruby version specification from the Gemfile to simplify our configuration?

Specifying the Ruby version in Gemfile was primarily useful for PaaS like Heroku.

Follow up for #2283.

BTW, given that our Ruby version is managed locally via `.ruby-version` and in production with Docker, Ruby version specification in `Gemfile` might be redundant.
Should we consider removing the Ruby version specification from the Gemfile to simplify our configuration?

Specifying the Ruby version in `Gemfile` was primarily useful for PaaS like Heroku.
Copy link

github-actions bot commented May 8, 2024

@github-actions github-actions bot added the reviewapps Build ReviewApp environment automatically if this label is granted label May 8, 2024

BUNDLED WITH
2.5.3
2.5.9
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

bundler v2.5.9 is the default gem version for ruby v3.3.1.

Copy link

github-actions bot commented May 8, 2024

Simplecov Report

Covered Threshold
60.89% 60%

@jacopen jacopen merged commit e0c7cc9 into main May 8, 2024
5 checks passed
@jacopen jacopen deleted the update_ruby_version branch May 8, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewapps Build ReviewApp environment automatically if this label is granted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants