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

Rails 6 Support #908

Merged
merged 4 commits into from
Aug 20, 2019
Merged

Rails 6 Support #908

merged 4 commits into from
Aug 20, 2019

Conversation

hone
Copy link
Member

@hone hone commented Aug 19, 2019

From @rafaelfranca on #907:

The current Rails 5 language pack work for Rails 6 applications but it could not be used because of the version constraints. This was making Rails 6 apps to be handled as generic rack apps and causing problems at deploy if no SECRET_KEY_BASE was set.

By adding a new class we can update the constraint so new Rails 6 apps can work as in the pre-releases.

In addition this adds a test.

rafaelfranca and others added 2 commits August 19, 2019 17:36
The current Rails 5 language pack work for Rails 6 applications but it
could not be used because of the version constraints. This was making
Rails 6 apps to be handled as generic rack apps and causing problems at
deploy if no SECRET_KEY_BASE was set.

By adding a new class we can update the constraint so new Rails 6 apps
can work as in the pre-releases.
@hone hone requested a review from schneems August 19, 2019 23:20
@schneems schneems merged commit 994f4aa into master Aug 20, 2019
@edmorley edmorley deleted the rails6 branch August 27, 2020 06:54
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.

3 participants