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

Allow setting an explicit database URL (in master) #93

Merged
merged 5 commits into from
Oct 6, 2016

Conversation

brockboland
Copy link
Contributor

These changes are all @chillpop commits from #92. I cherry-picked them into a new branch for merge into master.

As long as Travis doesn't have any issues with it, I'm going to merge this, since the code has already been reviewed and tweaked in that previous PR.

@vokal/ios-developers FYI

@brockboland
Copy link
Contributor Author

so that's how it's gonna be.

It's passing locally, I'll look into it.

@brockboland
Copy link
Contributor Author

Looks like Travis was just having a day: I re-ran the three jobs individually yesterday and they all passed. Just restarted the build to confirm they all still pass OK, and if so, I'll merge this.

@brockboland
Copy link
Contributor Author

Travis is definitely having some issues with Xcode 8: these just fail to build for no reason sometimes. I'll keep an eye on it in future PRs and see if there's anything that might need to change.

@brockboland brockboland merged commit a3770a5 into vokal:master Oct 6, 2016
@brockboland brockboland deleted the db_url branch October 6, 2016 14:52
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.

1 participant