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

Lock bigdecimal version to 3.1.4 or lower #50331

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

yahonda
Copy link
Member

@yahonda yahonda commented Dec 12, 2023

Motivation / Background

This commit addresses the Rails CI using Ruby master branch failure because Rails CI enables RAILS_STRICT_WARNINGS to raise RuntimeError for warnings.

https://buildkite.com/rails/rails/builds/102621#018c49df-e3de-4c2f-aeb7-6d8f08997da9/1102-1107

/rails/activesupport/lib/active_support/core_ext/big_decimal/conversions.rb:4: warning: bigdecimal/util is found in bigdecimal, which will no longer be part of the default gems since Ruby 3.4.0. Add bigdecimal to your Gemfile or gemspec. (RuntimeError)

Detail

According to https://bugs.ruby-lang.org/issues/20058 , This failure has been triggered since ruby/ruby@cc98265 and it will be addressed once the bigdecimal 3.1.5 is available at rubygems.org .

In the meantime, we can workaround this issue by logking bigdecimal version to 3.1.4 or lower. This commit should be reverted when the bigdecimal 3.1.5 is available at rubygems.org .

Additional information

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one change. Changes that are unrelated should be opened in separate PRs.
  • Commit message has a detailed description of what changed and why. If this PR fixes a related issue include it in the commit message. Ex: [Fix #issue-number]
  • Tests are added or updated if you fix a bug or add a feature.
  • CHANGELOG files are updated for the changed libraries if there is a behavior change or additional feature. Minor bug fixes and documentation changes should not be included.

This commit addresses the Rails CI using Ruby master branch failure
because Rails CI enables `RAILS_STRICT_WARNINGS` to raise RuntimeError for warnings.

https://buildkite.com/rails/rails/builds/102621#018c49df-e3de-4c2f-aeb7-6d8f08997da9/1102-1107
```ruby
/rails/activesupport/lib/active_support/core_ext/big_decimal/conversions.rb:4: warning: bigdecimal/util is found in bigdecimal, which will no longer be part of the default gems since Ruby 3.4.0. Add bigdecimal to your Gemfile or gemspec. (RuntimeError)
```

According to https://bugs.ruby-lang.org/issues/20058 ,
This failure has been triggered since ruby/ruby@cc98265
and it will be addressed once the bigdecimal 3.1.5 is available at rubygems.org .

In the meantime, we can workaround this issue by logking `bigdecimal` version to 3.1.4 or lower.
This commit should be reverted when the `bigdecimal` 3.1.5 is available at rubygems.org .
@yahonda
Copy link
Member Author

yahonda commented Dec 12, 2023

@yahonda yahonda merged commit 9715b86 into rails:main Dec 12, 2023
4 checks passed
@yahonda
Copy link
Member Author

yahonda commented Dec 14, 2023

Most of the railties failure has been addressed by https://bugs.ruby-lang.org/issues/20060

https://buildkite.com/rails/rails/builds/102772

@yahonda yahonda deleted the lock_bigdecimal_314 branch April 19, 2024 01:39
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.

None yet

1 participant