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

Support for Rails 8.0.0.beta1 #411

Open
navidemad opened this issue Oct 6, 2024 · 5 comments
Open

Support for Rails 8.0.0.beta1 #411

navidemad opened this issue Oct 6, 2024 · 5 comments

Comments

@navidemad
Copy link

Thanks for the awesome gem.
I recently migrated my app to Rails 8.0.0.beta1, and wanted to try this adapter but there is a lock becase activerecord-postgis-adapter >= 9.0.0 depends on activerecord ~> 7.1.0.
Looking forward to test it :)

@BuonOmo
Copy link
Member

BuonOmo commented Oct 7, 2024

Hi @navidemad

I don't think we'll do such a thing, as we are still trying to find time to handle 7.2 (non-beta) support. If you want to be the champion of it, or if you find a way to sponsor the gem (https://opencollective.com/rgeo), then this may happen :)

We'll for sure take care of it when it goes out of beta (I'm also paid maintainer of the activerecord-cockroachdb-adapter, hence I usually first upgrade there, and then here)

@formigarafa
Copy link
Contributor

I've been looking around and trying to help but I am a bit confused because of the current state of master(or main) branch.
There is #397 merged titled "feat!: ActiveRecord 7.2 support". There are some issues and PRs mentioning something around jRuby which I could not understand what is the decision moving forward. Others talking about specs breaking but only in CI.
And I could not figure what is related to 7.2 or what exactly is breaking.
I noticed that in the history of the project people have used some labels for active record versions. It might be helpful having some of these labels applied for engaging the community.
If I could see some direction, steps, checklist,... but I couldn't figure what is needed.

@BuonOmo
Copy link
Member

BuonOmo commented Oct 11, 2024

@formigarafa currently we have only partial support of the beta, and I have a PR ongoing for full support, but tests are failing (#405).

So the current goal is merging #405, which I'm considering to merge with some tests ignored and opening related issues. If that is the case would you be able to help on fixing tests (and thus maybe some parts of the codebase as well...).

About JRuby support, we're not really supporting it with new rails versions, I think @seuros knows why. But anyway, we should already focus on having the main C ruby up before thinking jruby!

@seuros
Copy link
Contributor

seuros commented Oct 11, 2024

Currently, we do not support JRuby because the latest version of the gem only supports a single version of ActiveRecord (7.2.x).

Screenshot 2024-10-11 at 13 03 17

Unlike CRuby, JRuby does not use the pg gem but relies on a Java-compliant adapter. As shown in the matrix above, the latest supported version is Rails 7.1.x, which is an unreleased version and has a few issues in the tracker.

I believe we could add JRuby support once it is accompanied by appropriate tests. The last version of this gem that I have personally tested is with Rails 6.x.

@formigarafa
Copy link
Contributor

That clarify heaps, thanks. I was wondering if jRuby compatibility was a blocker. If that isn't the case, I agree with @BuonOmo, just focus on what is supported in first place should be the best for everyone. Even better with a single PR to look at. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants