Skip to content

Support Rails 8.1 - #250

Open
agrare wants to merge 1 commit into
ManageIQ:masterfrom
agrare:test_rails_8_1
Open

agrare wants to merge 1 commit into
ManageIQ:masterfrom
agrare:test_rails_8_1

Conversation

@agrare

@agrare agrare commented Aug 11, 2026 •

Copy link
Copy Markdown
Member

Add support for rails 8.1. Even though no changes for 8.1 were required it is probably better to stick with keeping this version 1:1 with rails minor version.

@kbrock

kbrock commented Aug 12, 2026

Copy link
Copy Markdown
Member

Seriously? This is working?

I swung by to see if this needed some love, but it looks like we left it in a decent spot. Think no major changes from 7.2 to here

@agrare

agrare commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

Seriously? This is working?

I know I was shocked, not going to lie my heart dropped a little when I saw this one in the list to update but seems to be all good.

@miq-bot

miq-bot commented Aug 14, 2026

Copy link
Copy Markdown
Member

Checked commit agrare@330d622 with ruby 3.3.10, rubocop 1.88.2, haml-lint 0.76.0, and yamllint 1.37.1
0 files checked, 0 offenses detected
Everything looks fine. 👍

@kbrock

kbrock commented Aug 25, 2026

Copy link
Copy Markdown
Member

reminder:

This repo follows a rails centric version numbering scheme and not SemanticVer.

For previous efforts, internal apis change across boundaries. so we based the major version upon the rails major/minor (not bug) version number. I copied the way that jruby handled their rails compatibility gems.

There hasn't been much change in ActiveRecord internals for a while. Not sure if that means people are less reckless or there hasn't been much progress? Maybe things have finally stabilized or maybe it is due for a major internal change - time will only tell.

Either way, I suggest sticking with rails centric version numbering, but do what ever is the easiest to maintain.

@agrare agrare changed the title [WIP] Test rails 8.1 Support Rails 8.1 Oct 13, 2026
@agrare agrare removed the wip Work in progress label Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants