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

Remove Rails v5.2 support #4850

Conversation

waiting-for-dev
Copy link
Contributor

Summary

Rails v5.2 is EOL since 01 Jun 2022.

Our new release policy makes it clear that we don't support Rails versions that are no longer maintained.

We also remove the indirection to set current host for ActiveStorage. As we only need to include a Rails module, there's neither the need to go through a custom module.

Closes #4842

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

  • I have written a thorough PR description.
  • I have kept my commits small and atomic.
  • I have used clear, explanatory commit messages.

The following are not always needed (cross them out if they are not):

  • [ ] I have added automated tests to cover my changes.
  • [ ] I have attached screenshots to demo visual changes.
  • [ ] I have opened a PR to update the guides.
  • [ ] I have updated the README to account for my changes.

Rails v5.2 is EOL since 01 Jun 2022 [1].

Our new release policy [2] makes it clear that we don't support Rails
versions that are no longer maintained.

[1] - https://endoflife.date/rails
[2] - https://solidus.io/release_policy/
@waiting-for-dev waiting-for-dev requested a review from a team as a code owner January 12, 2023 10:25
@github-actions github-actions bot added changelog:solidus_api Changes to the solidus_api gem changelog:solidus_core Changes to the solidus_core gem labels Jan 12, 2023
@waiting-for-dev waiting-for-dev force-pushed the waiting-for-dev/remove_rails_v5.2_support branch from 813a6f1 to 13d31e8 Compare January 12, 2023 11:43
@waiting-for-dev waiting-for-dev force-pushed the waiting-for-dev/remove_rails_v5.2_support branch from 13d31e8 to aea7fd0 Compare January 13, 2023 04:49
We no longer support Rails v5.2, so there's no need to support the old
way [1]. As we only need to include a Rails module, there's neither the need
to go through a custom module.

[1] - rails/rails@e33c3cd
@waiting-for-dev waiting-for-dev force-pushed the waiting-for-dev/remove_rails_v5.2_support branch from aea7fd0 to a3119d5 Compare January 13, 2023 10:07
@waiting-for-dev waiting-for-dev merged commit a71107b into solidusio:master Jan 16, 2023
@waiting-for-dev waiting-for-dev deleted the waiting-for-dev/remove_rails_v5.2_support branch January 16, 2023 05:09
@tvdeyen
Copy link
Member

tvdeyen commented Nov 1, 2023

This PR is missing the activestorage dependency. I added it in #5479

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_api Changes to the solidus_api gem changelog:solidus_core Changes to the solidus_core gem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove support for Rails 5.2
4 participants