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

Always use sqlite as data source #292

Merged
merged 1 commit into from
Apr 23, 2020
Merged

Conversation

orangejulius
Copy link
Member

BREAKING CHANGE: This implements a hard switchover to using SQLite files
for admin lookup. Using bundle files is no longer supported, and as a
result the imports.whosonfirst.sqlite setting is now ignored.

wof-admin-lookup will now behave as if that flag was always set to
true.

This PR requires pelias/whosonfirst#498 to be merged first

@orangejulius
Copy link
Member Author

@Joxit I just wanted to say, given I have now spent a ton of time undoing it all, how much work you clearly put into making the tests in this repo support both bundles and SQLite back in #266.

It's much appreciated even if we are now in a glorious SQLite-only future :) 🙏

BREAKING CHANGE: This implements a hard switchover to using SQLite files
for admin lookup. Using bundle files is no longer supported, and as a
result the `imports.whosonfirst.sqlite` setting is now ignored.

`wof-admin-lookup` will now behave as if that flag was always set to
true.

Closes #293
@orangejulius orangejulius merged commit bd31b26 into master Apr 23, 2020
@orangejulius orangejulius deleted the drop-bundle-support branch April 23, 2020 20:16
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