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

Downgrade ActiveSupport dependency to >= 6.1 #354

Closed

Conversation

igor-alexandrov
Copy link
Contributor

In this pull request, I propose downgrading the ActiveSupport dependency in MRSK to a minimum version of 6.1. Since no ActiveSupport 7.0 methods are utilized in MRSK, this change allows for broader compatibility with Rails 6 applications.

By supporting the current Rails version as well as the previous one, MRSK becomes an even more widely used deployment tool.

@dhh
Copy link
Member

dhh commented Jun 26, 2023

I'd rather not go down this path, because MRSK needn't be added to the application's Gemfile, so there's no demand for compatibility. Maybe we need to document this better. We've been using MRSK with Rails 2.3 applications! Install mrsk as a global, then you're good to go. Please do have a look at what tweaks we need to the docs to make this clearer.

@dhh dhh closed this Jun 26, 2023
@igor-alexandrov
Copy link
Contributor Author

Good point, will work on documentation.

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

Successfully merging this pull request may close these issues.

None yet

3 participants