Rails 5 Support#10
Open
michihuber wants to merge 6 commits into
Open
Conversation
michihuber
commented
Mar 16, 2016
michihuber
commented
Contributor
- Drops Rails 3.2 and 4.0 support
- Bumps version to 0.10.0
Drops support for Rails 3.2 and Rails 4.1.
- gemspec: relax version constraint from < 5.1 to < 6 - ConnectionHandler: add flush_idle_connections! (new in Rails 5.2) - ConnectionHandler: guard against Rails 5.2 passing spec name strings instead of classes to retrieve_connection_pool - postamt.rb: override retrieve_connection on AR::Base to restore class-based routing (required for per-model default_connection) - Relation#delete_all: remove conditions param (dropped in Rails 5.0) - LogSubscriber: prefer event.payload[:connection] over ObjectSpace._id2ref - Add testapp52 with Rails 5.2 test app and Nix dev environment (flake.nix)
- Keep testapp50/ and testapp52/ (our renames of testapp42) - Drop testapp/ (master's conflicting rename of testapp42) - Take master's railtie.rb fix: use parse_task_string for Rake task detection - Update README to reference testapp52 bin/setup workflow
- testapp50/bin/setup: init/start postgres, create roles, load schema, grant slave SELECT-only; mirrors testapp52/bin/setup - Both testapps use port 5433 and repo-root postgres/data directory - testapp50: pin pg < 1.6, remove file_watcher (needs listen gem), fix merge conflict in test.rb, remove fixtures :all (slave is read-only) - Move postgres/ to root .gitignore
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.