Skip to content

Update Gemfile.lock for Ruby 3.3.0#50516

Merged
dhh merged 1 commit intorails:mainfrom
skipkayhil:hm-bundle-update
Jan 2, 2024
Merged

Update Gemfile.lock for Ruby 3.3.0#50516
dhh merged 1 commit intorails:mainfrom
skipkayhil:hm-bundle-update

Conversation

@skipkayhil
Copy link
Member

@skipkayhil skipkayhil commented Jan 1, 2024

Based on #50515 Merged 👍


This commit is meant to accomplish two things:

  • update gems with precompiled C extensions (nokogiri) so that Rails'
    lockfile can be bundle installd on Ruby 3.3
  • remove some Gemfile conditionals for Ruby < 3 now that 3.1 is the
    minimum version

Additionally, a requirement is added to ignore mdl 0.13.0 because it
raises false positive warnings. The commit introducing those warnings
has been reverted but there has not been a release since the revert.

@dhh
Copy link
Member

dhh commented Jan 2, 2024

Needs a rebase

This commit is meant to accomplish two things:
- update gems with precompiled C extensions (nokogiri) so that Rails'
  lockfile can be `bundle install`d on Ruby 3.3
- remove some Gemfile conditionals for Ruby < 3 now that 3.1 is the
  minimum version

Additionally, a requirement is added to ignore mdl 0.13.0 because it
raises false positive warnings. The commit introducing those warnings
has been reverted but there has not been a release since the revert.
@dhh
Copy link
Member

dhh commented Jan 2, 2024

A GREEN BUILD! Been a long time since I've seen that 😂

@dhh dhh merged commit 66c1745 into rails:main Jan 2, 2024
@skipkayhil skipkayhil deleted the hm-bundle-update branch January 2, 2024 15:54
yoones pushed a commit to yoones/rails that referenced this pull request Mar 6, 2025
This commit is meant to accomplish two things:
- update gems with precompiled C extensions (nokogiri) so that Rails'
  lockfile can be `bundle install`d on Ruby 3.3
- remove some Gemfile conditionals for Ruby < 3 now that 3.1 is the
  minimum version

Additionally, a requirement is added to ignore mdl 0.13.0 because it
raises false positive warnings. The commit introducing those warnings
has been reverted but there has not been a release since the revert.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants