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

Enforce minimal web-console version for Rails 5 #25907

Merged
merged 1 commit into from
Jul 21, 2016

Conversation

gsamokovarov
Copy link
Contributor

I dropped the version constraint in web-console with the idea it will be
easier to upgrade the console between Rails releases. However, issues
like #25899 started popping up.

I'm reintroducing the constraint, but this time, I don't set an upper
limit to the major version. This will keep the web-console in a version
that always works for the current Rails and can be easily upgraded
to the last one with bundle update.

We may need to backport this for Rails 5.0.1.

Fixes #25899.

I dropped the version constraint in web-console with the idea it will be
easier to upgrade the console between Rails releases. However, issues
like rails#25899 started popping up.

I'm reintroducing the constraint, but this time, I don't set an upper
limit to the major version. This will keep the web-console in a version
that always works for the current Rails version and can be easily
upgraded to the last one with `bundle update`.

We may need to backport this for Rails 5.0.1.

Fixes rails#25899.
@rails-bot
Copy link

r? @senny

(@rails-bot has picked a reviewer for you, use r? to override)

@senny senny merged commit 0800e5d into rails:master Jul 21, 2016
senny added a commit that referenced this pull request Jul 21, 2016
Enforce minimal web-console version for Rails 5
@senny
Copy link
Member

senny commented Jul 21, 2016

@gsamokovarov thanks. I backported this to 5-0-stable (0304852)

@gsamokovarov
Copy link
Contributor Author

Thanks @senny!

@maclover7
Copy link
Contributor

maclover7 commented Jul 21, 2016

Thank you @gsamokovarov! <3

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

Successfully merging this pull request may close these issues.

None yet

4 participants