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

Add Rails 6 support #329

Merged
merged 6 commits into from
May 14, 2019
Merged

Add Rails 6 support #329

merged 6 commits into from
May 14, 2019

Conversation

kerrizor
Copy link
Collaborator

Adds Rails 6 testing, but official support is still considered experimental until an official 6.0.0 release.

Also bumping the version of sqlite3, as AR 6 requires ~> 1.4
Slightly annoying, but I miss simple navigation in my debugger..
After each test, we teardown the in-memory AR DB/cache classes so we can
start with a fresh AR object graph. AR 6 changed some of the internals
(rails/rails@1b2efe5)
around detecting what should be unloaded when
ActiveSupport::DescendantsTracker#clear is called; this is the danger on
relying on internals! This little hack manually removes the
ActiveRecord::Base descendants from the class variable that tracks
what's loaded and what isn't; It will probably live on for years.
@kerrizor kerrizor force-pushed the kerrizor/add-rails-6-support branch from 32c2942 to 77a2c9f Compare May 14, 2019 00:35
@kerrizor kerrizor merged commit 360fc51 into master May 14, 2019
@kerrizor kerrizor deleted the kerrizor/add-rails-6-support branch May 14, 2019 01:16
@kerrizor kerrizor mentioned this pull request May 14, 2019
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

1 participant