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

Upgrade to Ruby 3.4.1 #937

Merged
merged 6 commits into from
Dec 27, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .ruby-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3.3.5
3.4.1
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
ARG RUBY_VERSION=3.3.5
ARG RUBY_VERSION=3.4.1
ARG NODE_VERSION=22.7.0

# Use Node image so we can pull the binaries from here.
Expand Down Expand Up @@ -37,7 +37,7 @@ RUN mkdir -p $APP_HOME
WORKDIR $APP_HOME

# Copy dependencies files and install libraries.
COPY --link Gemfile Gemfile.lock package.json yarn.lock .yarnrc.yml ./
COPY --link Gemfile Gemfile.lock package.json yarn.lock .yarnrc.yml .ruby-version ./

RUN corepack enable
RUN gem install bundler && bundle install -j 4 && yarn install --immutable && \
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.dev
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
ARG RUBY_VERSION=3.3.5
ARG RUBY_VERSION=3.4.1
ARG NODE_VERSION=22.7.0

# Use Node image so we can pull the binaries from here.
Expand Down
2 changes: 1 addition & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

source 'https://rubygems.org'

ruby '~> 3.3.5'
ruby file: '.ruby-version'

gem 'rails', '~> 7.2.2'

Expand Down
14 changes: 1 addition & 13 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -243,8 +243,6 @@ GEM
faraday-retry (2.2.1)
faraday (~> 2.0)
ffi (1.17.0)
ffi (1.17.0-arm64-darwin)
ffi (1.17.0-x86_64-linux-gnu)
flipper (1.3.2)
concurrent-ruby (< 2)
flipper-active_record (1.3.2)
Expand All @@ -265,12 +263,6 @@ GEM
google-protobuf (4.29.2)
bigdecimal
rake (>= 13)
google-protobuf (4.29.2-arm64-darwin)
bigdecimal
rake (>= 13)
google-protobuf (4.29.2-x86_64-linux)
bigdecimal
rake (>= 13)
Comment on lines -268 to -273
Copy link
Member

Choose a reason for hiding this comment

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

hmmm strange that these gems for these platforms got removed 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is odd. I try to revert this changes but bundle install keeps doing it.

When it fails, on GH actions it says :

ffi-1.17.0-x86_64-linux-gnu requires ruby version < 3.4.dev, >= 2.5, which is
incompatible with the current version, 3.4.1

Maybe is no more needed? I couldn't find anything on google 😂

Copy link
Member

Choose a reason for hiding this comment

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

😮 ok. Let's go ahead and if something breaks we can fix it

has_scope (0.8.2)
actionpack (>= 5.2)
activesupport (>= 5.2)
Expand Down Expand Up @@ -369,10 +361,6 @@ GEM
nokogiri (1.17.2)
mini_portile2 (~> 2.8.2)
racc (~> 1.4)
nokogiri (1.17.2-arm64-darwin)
racc (~> 1.4)
nokogiri (1.17.2-x86_64-linux)
racc (~> 1.4)
octokit (9.2.0)
faraday (>= 1, < 3)
sawyer (~> 0.9)
Expand Down Expand Up @@ -680,7 +668,7 @@ DEPENDENCIES
yaaf (~> 3.0)

RUBY VERSION
ruby 3.3.5p100
ruby 3.4.1p0

BUNDLED WITH
2.5.18
Loading