Skip to content

Conversation

@pirj
Copy link
Member

@pirj pirj commented Jan 22, 2022

This:

  • fixes 3.0 that were actually running against Ruby 3.2-dev
  • Rails master that is now Rails 7.1
  • moves JRuby builds from Travis to GHA
  • brings bundler update step back (Get RubyGems/Bundler update CI step back #2549)
  • fixes argument matching for enqueued_mail

@pirj pirj self-assigned this Jan 22, 2022
@pirj pirj mentioned this pull request Jan 22, 2022
21 tasks
@pirj pirj requested a review from JonRowe January 22, 2022 21:34
@pirj pirj force-pushed the add-ruby-3.1-support-to-rspec-rails-5.0 branch from 122b4a8 to 347a9e6 Compare January 22, 2022 22:28
pirj and others added 6 commits January 23, 2022 02:09
rubygems/bundler#6728

ailure/Error: expect(`bundle exec #{script} 2>&1`).to be_empty
       expected `"Your RubyGems version (2.7.6.3)) has a bug that prevents `required_ruby_version` from working for Bundler. Any scripts that use `gem install bundler` will break as soon as Bundler drops support for your Ruby version. Please upgrade RubyGems to avoid future breakage and silence this warning by running `gem update --system 3.2.3`\n".empty?` to be truthy, got false
`rspec-rails` 6.0 will support Rails 7
Add additional logic to the ActionMailer argument parsing to accomodate for differences under Ruby 3.1/Rails 6.1

Co-authored-by: Fabio Napoleoni <[email protected]>
Co-authored-by: Phil Pirozhkov <[email protected]>
@pirj pirj force-pushed the add-ruby-3.1-support-to-rspec-rails-5.0 branch from 21b462c to b020050 Compare January 22, 2022 23:10
Exclude both the "bundle" and "tmp" directory.  The former ensures we don't run rubocop against installed gems.  The latter ensures we don't run it against generated code and apps locally.
@pirj pirj merged commit 5feb2c7 into 5-0-maintenance Jan 22, 2022
@pirj pirj deleted the add-ruby-3.1-support-to-rspec-rails-5.0 branch January 22, 2022 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants