rubyonjets/jets

Support for activerecord/activesupport 6.0

Closed this issue · 4 comments

Summary

Hi! First of all, thanks for these great framework!
I would like suggest some alternative to support use Jets with activerecord/activesupport bellow 6.1.0. For exemple: 6.0.0

Motivation

There is a lot of rubygems that normally depends from activesupport, activerecord and do not support the last versions from that gems. Eg: I need to use the redshift-client gem, but it require to use activerecord 6.0.0 or bellow.

Drawbacks

I understand that support really old versions will cost. But, at least, 2 latest versions maybe is a good investment in order to allow more people to use Jets.

Wondering which gem examples require 6.0 and below.

That being said, think it's ok to loosen the activesupport and the like to 6.0+ instead of 6.1+ in the jets.gemspec. Though, remember there were some interfaces that changed from 6.0 to 6.1. Here's one example #539 There may be more.

So would need some respond_to? to check for what interface to use. Hence the overhead. Think it's worth investigating. Will review and consider PRs. Of course, no sweat either way 👍

Hi @tongueroo! Thanks for the update!

Actually, there is no explicit dependency.
But in the "activerecord6-redshift-adapter" gem for example, the class active_record/connection_adapters/determine_if_preparable_visitor.rb is used in order to extend a module.
And that class do not exists in the ActiveRecord 6.1 version.

Is there some code to change in Jets in order to support 6.0+ ? If so, is there some path?

Hi @tongueroo

Is there some code to change in Jets in order to support 6.0+ ? If so, is there some path?

@rinaldifonseca I am curious to hear out what exact issue you have with activesupport 6.0 ?

Have you tried using jets v2.3.19 ? The requirement to 6.1 was done with 3.0 jets release
or you could also fork jet and relax the requirement back to 6.0