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.

Sign up for GitHub

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

Jump to bottom

Sqlite compatibility #438

Merged
merged 1 commit into from
Mar 23, 2016
Merged

Sqlite compatibility #438

merged 1 commit into from
Mar 23, 2016

Conversation

jywarren
Copy link
Member

Make sure these boxes are checked before your pull request is ready to be reviewed and merged. Thanks!

  • tests pass -- rake test
  • code has been rebased on top of latest master (check if another pull request was added recently, and please rebase)
  • pull request is descriptively named and, if possible, multiple commits squashed if they're smaller changes
  • reviewed/confirmed/tested by another contributor or maintainer

Thanks!

@jywarren jywarren force-pushed the sqlite branch 2 times, most recently from fbeef54 to 407d7e3 Compare March 21, 2016 22:18
This was referenced Mar 22, 2016
Closed
Closed
@jywarren jywarren force-pushed the sqlite branch 3 times, most recently from 2808177 to de41469 Compare March 23, 2016 16:24
@jywarren
Copy link
Member Author

Running rake test on this in cloud9 passes, but there are lots of unrelated external warnings from the gems: Moved this to a new issue, at #440

@jywarren
initial steps to remove spatial_adapter
39f55b2 
initial stab at sqlite compatibility and spatial_adapter deprecation

passing tests on sqlite now

README and Gemfile cleanup

README login details

tweaked cloudnine install script

updated cloud9 database config

additional install script updates

update to README for cloudnine and cloudnine install script tweaks

added db/schema.rb.example and modified gitignore

readme update

additional install documentation in README
@jywarren jywarren mentioned this pull request Mar 23, 2016
Closed
@jywarren jywarren merged commit 95bf48c into publiclab:master Mar 23, 2016
@jywarren jywarren mentioned this pull request Mar 23, 2016
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jywarren