-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
Support Active Record 4.2 WIP #26
base: master
Are you sure you want to change the base?
Conversation
d8fdf76
to
e40987d
Compare
@dschweisguth I'd like to propose changes in baby steps, let's first add compatibility with newer rgeo , merge, and then let's focus on ActiveRecord 4.2 Also, things like code cleanup and .gitignore / Gemfile improvements we can do in a separate PR. It'll be easier to handle this and we'll deliver results in smaller increments, faster. |
I based this on the branch for my previous PR. Only the last commit here will remain after that other branch is merged and this one is rebased. |
e40987d
to
e10e441
Compare
f2fc87a
to
f14b07f
Compare
This commit reapplies fjl82 and agoln's commits from fjl82/activerecord-mysql2spatial-adapter to current master. Not all tests pass yet.
f14b07f
to
96e6ec7
Compare
Hi, Any news ? |
I think it's time to find a new maintainer for this project, I have not used this in my work for a long time. I'll look at your new PR soon |
Other things are taking my time right now. I'll eventually get back to the project that uses this gem, but not very soon. Please do tell me if I should rebase anything of mine or test new releases of this gem against my project. |
This PR intends to fix #12. It starts with a reapplication of the changes in https://github.com/fjl82/activerecord-mysql2spatial-adapter to current master.
To do: