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

Bump activerecord dependency for Rails 6.1 #503

Merged
merged 2 commits into from
Dec 14, 2020

Conversation

joergschiller
Copy link
Contributor

Allows using paranoia with Rails 6.1. This is a follow up of #501 and bumps the dependency in the gemspec, adds Rails 6.1 to the build matrix, and fixes a deprecation message.

@walski wrote in #501 (comment):

I'm happy to close this PR in favor of a new one from @joergschiller

Calling `<<` to an ActiveModel::Errors message array in order to add an error is deprecated. Please call `ActiveModel::Errors#add` instead.
@joergschiller
Copy link
Contributor Author

@sevenseacat I really appreciate your work on maintaining this project (at least it looks like you merged the last pull requests). Is there anything I can do to make you reviewing this PR easier?

@sevenseacat sevenseacat merged commit 15a09a8 into rubysherpas:core Dec 14, 2020
@sevenseacat
Copy link
Member

It looks like there also hasn't been a new release in some time - I'll try to work out how to make that happen now that we technically support Rails 6.1!

@koleksiuk
Copy link

@sevenseacat when can we expect the release for 6.1 support?

@sevenseacat
Copy link
Member

@koleksiuk when I get some time to figure out how to do it! :D

@sevenseacat
Copy link
Member

@koleksiuk I think I've done it right. v2.4.3 should be good to go.

@koleksiuk
Copy link

Thank you!

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.

3 participants