I think latest master could bump requirement to Rails 6 for next release as Rails 5.2 is supposed to lose support within the next 2 weeks
w
waiting_for_dev
06/16/2022, 3:51 AM
Hey @JDutil, thanks for your suggestion. Yeah, we're going to deprecate Rails 5.2 on Solidus v3.2, and discontinue support on v4.0
👍 1
j
JDutil
06/16/2022, 3:53 AM
Cool, perhaps start a rails 6 branch if there isn't one already?
w
waiting_for_dev
06/16/2022, 4:18 AM
What do you mean by a new branch? Rails 6 & 7 are already supported (7 only on master for now)
👍 1
t
tvdeyen
06/16/2022, 1:07 PM
I am not sure there is a need to deprecate Rails 5.2 support. Just remove it in 3.2. People still using Rails 5.2 are most likely not running 3.x anyways.
j
JDutil
06/17/2022, 2:00 AM
Meant a branch that can adopt the features Ben wanted to use in the meantime
w
waiting_for_dev
06/17/2022, 3:43 AM
Rails 7 is supported on master, so Rails 7 features can be used there. It'll be available on v3.2 very soon.
j
JDutil
06/20/2022, 12:11 AM
I should have said a branch that removes support for old features in favor of new ones like the bulk import Ben was mentioning. It doesn't really matter to me though just throwing the idea out there if it'd help Ben get the contributions accepted in the future