-
Notifications
You must be signed in to change notification settings - Fork 388
Add support for Rails 7.1 and 7.2 #705
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
Conversation
spec/configuration_spec.rb
Outdated
@@ -138,7 +138,7 @@ | |||
|
|||
context "when passed an invalid value" do | |||
it "should raise an ArgumentError" do | |||
expect(lambda{MoneyRails.rounding_mode = "booyakasha"}).to raise_error(ArgumentError, 'booyakasha is not a valid rounding mode') | |||
expect { MoneyRails.rounding_mode = "booyakasha" }.to raise_error(ArgumentError, 'booyakasha is not a valid rounding mode') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was an RSpec deprecation warning
@schmijos would you mind rebasing your branch with |
c10ed9b
to
b92db01
Compare
b92db01
to
9c5a4bf
Compare
diff --git a/gemfiles/rails7.1.gemfile b/gemfiles/rails7.1.gemfile new file mode 100644 index 0000000..3951f70 --- /dev/null +++ b/gemfiles/rails7.1.gemfile @@ -0,0 +1,17 @@ +source 'https://rubygems.org' + +gem 'rails', '~> 7.1.0' +gem 'sprockets-rails' +gem 'i18n', '~> 1.6' + +platforms :jruby do + gem "activerecord-jdbc-adapter" + gem "activerecord-jdbcsqlite3-adapter" + gem "jruby-openssl" +end + +platforms :ruby do + gem "sqlite3", "~> 1.4" +end + +gemspec path: '../' diff --git a/gemfiles/rails7.2.gemfile b/gemfiles/rails7.2.gemfile new file mode 100644 index 0000000..7050731 --- /dev/null +++ b/gemfiles/rails7.2.gemfile @@ -0,0 +1,17 @@ +source 'https://rubygems.org' + +gem 'rails', '~> 7.2.0' +gem 'sprockets-rails' +gem 'i18n', '~> 1.6' + +platforms :jruby do + gem "activerecord-jdbc-adapter" + gem "activerecord-jdbcsqlite3-adapter" + gem "jruby-openssl" +end + +platforms :ruby do + gem "sqlite3", "~> 2.0" +end + +gemspec path: '../'
9c5a4bf
to
d917547
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@schmijos I just removed the Ruby 3.0 test and it's all green now 😊 Based on the discussion here we will just drop support for Ruby 3.0 since the new money-rails
version will depend on the newest money
version and that requires at least 3.1 ✌🏻
I might simplify a bit the new gemspecs you added based on the changes I've done [here (https://github.com//pull/719) but I am merging your PR as it is!
This adds two new Gemfiles and bumps minimal versions.