FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Support Rails 8.1 by zhuravel · Pull Request #782 · binarylogic/authlogic · GitHub

/ authlogic Public

Support Rails 8.1 - #782

Merged
binarylogic merged 4 commits into
binarylogic:masterfrom
talkable:rails-8.1
Nov 13, 2025
ConversationCommits (4)ChecksFiles changed
Merged

Support Rails 8.1#782
binarylogic merged 4 commits into
binarylogic:masterfrom
talkable:rails-8.1

Conversation

zhuravel commented Oct 16, 2025

Copy link
Copy Markdown
Contributor

Changes:

  • Added support for Rails 8.1
  • Removed support for EOL Ruby versions (< 3.2)
  • Removed support for EOL Rails versions (< 7.2)
  • Fixed badge in README.md that was not updated after migration from Travis CI
  • Replaced Module.send :include with Module.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:

gem "authlogic", github: "binarylogic/authlogic", ref: "refs/pull/782/head"

ashkulz commented Oct 25, 2025

Copy link
Copy Markdown

Would appreciate your merging this and making a release, @binarylogic 🙂

brendon commented Oct 27, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Thanks @zhuravel, great to see this all updated :)

ashkulz commented Oct 28, 2025

Copy link
Copy Markdown

@brendon the author of the PR is @zhuravel, I'm just another bystander waiting for it to get merged 🙂

mihael added a commit to mihael/authlogic that referenced this pull request Nov 10, 2025
Support Rails 8.1
9a8f584 
- 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

kryali commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

@binarylogic any chance we can get this PR merged and released since rails 8.1 has been released weeks ago?

brendon commented Nov 13, 2025

Copy link
Copy Markdown
Collaborator

@binarylogic, if you need any assistance with maintainership of this gem let me know. I'm the maintainer of acts_as_list, ranked-model and my own positioning and have recently joined together with another couple of devs to assist with maintaining sorcery. I'd be happy to look at PR's like this and get them merged provided you'd be keen to do the gem releases?

binarylogic merged commit 9df840c into binarylogic:master Nov 13, 2025
10 checks passed

binarylogic commented Nov 13, 2025

Copy link
Copy Markdown
Owner

Thanks everyone, apologies for the delay. @brendon that would be great! I will email you now.

ashkulz commented Nov 14, 2025

Copy link
Copy Markdown

Can we have a release to RubyGems, please?

brendon commented Nov 16, 2025

Copy link
Copy Markdown
Collaborator

Yep, all good. Just waiting for @binarylogic to fix up a rubygems issue for me and then I can publish it :)

randikabanura commented Nov 21, 2025 •
edited
Loading

Copy link
Copy Markdown

Hi @binarylogic,
Any chance you could cut a new release quickly when you get a sec? Our application upgrade’s waiting on it. Thanks a ton! 🙏

brendon commented Nov 27, 2025

Copy link
Copy Markdown
Collaborator

No luck contacting @binarylogic for now, but I've released 6.6.0 using my other Rubygems account that was linked accidentally. I'll keep attempting to get the account situation sorted. Thanks for the hard work @zhuravel :)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

brendon brendon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

Back | FazBrowse Home | New Git URL