-
Notifications
You must be signed in to change notification settings - Fork 636
Support Rails 8.1 #782
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
Support Rails 8.1 #782
Conversation
|
Would appreciate your merging this and making a release, @binarylogic 🙂 |
|
Thanks @zhuravel, great to see this all updated :) |
- Expand AR/AS/AM bounds to < 8.3 in gemspec - Normalize cookie SameSite: accept symbols/strings; write symbols to Rails cookie jar; keep strings for test mock - Add Rails 8.1 gemfiles (full and sqlite-only) and 8.2 stubs - Add Rails 8.1 to CI matrix - Update README compatibility and contributing docs - Refactor: use Module#include instead of send(:include) for AR hooks, Sinatra, and TestCase (aligns with upstream PR binarylogic#782) Refs: binarylogic#782
|
@binarylogic any chance we can get this PR merged and released since rails 8.1 has been released weeks ago? |
|
@binarylogic, if you need any assistance with maintainership of this gem let me know. I'm the maintainer of |
|
Thanks everyone, apologies for the delay. @brendon that would be great! I will email you now. |
|
Can we have a release to RubyGems, please? |
Changes:
Module.send :includewithModule.include(method is public since Ruby 2.1 released in 2013)For those who need to use this fix before the PR is merged, you can add the gem to your Gemfile like this: