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 Rspec to ~5.0; support Rails 6.1 #41

Merged

Conversation

henrypoydar
Copy link
Contributor

Why?

To use this gem with a Rails 6.1 application

What?

Bumps development environment to Ruby 2.7.2. from Ruby 2.6.2
Upgrades rspec-rails from 3.0 to 5.0
Bumps upper limit on Rails support from 6.0 to 6.1

Testing Notes

All specs pass, but did not test with earlier Rails < 6.0. Recommend a minor release instead of a patch release, and bumping up the minimum Rails version to 6.0.

@@ -10,5 +10,5 @@ gemspec
# Git. Remember to move these dependencies to your gemspec before releasing
# your gem to rubygems.org.

gem "rails", "~> 6.0"
gem "rails", "~> 6.1", ">= 6.0"
Copy link

Choose a reason for hiding this comment

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

Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.

Copy link
Contributor

Choose a reason for hiding this comment

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

Bad boy hound 😆 🐕 I must need to adjust the settings because it should be looking for double quotes; so this is just fine.

@rreinhardt9 rreinhardt9 self-requested a review March 25, 2021 17:34
@rreinhardt9
Copy link
Contributor

Hi @henrypoydar !!! Thanks so much for this PR; really appreciate it 🙌🏻 . I'll check it out and work on get back with ya if there is any feedback. Looks pretty straightforward 🎉

Copy link
Contributor

@rreinhardt9 rreinhardt9 left a comment

Choose a reason for hiding this comment

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

This all looks great to me!! The build for Rails 5 and Ruby 2.6 are both still green in Travis as well as the builds for Rails 6 and Ruby 2.7.

Really appreciate your help @henrypoydar !

@@ -10,5 +10,5 @@ gemspec
# Git. Remember to move these dependencies to your gemspec before releasing
# your gem to rubygems.org.

gem "rails", "~> 6.0"
gem "rails", "~> 6.1", ">= 6.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Bad boy hound 😆 🐕 I must need to adjust the settings because it should be looking for double quotes; so this is just fine.

@rreinhardt9 rreinhardt9 merged commit 1e9ce33 into lessonly:master Mar 25, 2021
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