-
Notifications
You must be signed in to change notification settings - Fork 120
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
Stop using tags-> in YAML #1199
Milestone
Comments
Amen!
… On Apr 7, 2017, at 03:49, Matt Amos ***@***.***> wrote:
Since we moved to the hstore schema, everything is in the tags column and the tags-> prefix is unnecessary. We've got a mix of different uses in the YAML files, but it could be confusing for any new contributors whether tags-> is required or not (just to be clear: it's not).
This should just be a sed 's/tags->//' on the YAML files, but it's possible that it might throw up some unexpected regressions.
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@zerebubuth @rmarianski Does your RAWR tile refactor address this issue as process changed? |
I think it's a good idea to fit into the rawr tiles work. |
There are still a few of these hanging out. Deferring to next milestone. |
Fixed by #1771. |
QA verified. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Since we moved to the hstore schema, everything is in the tags column and the
tags->
prefix is unnecessary. We've got a mix of different uses in the YAML files, but it could be confusing for any new contributors whethertags->
is required or not (just to be clear: it's not).This should just be a
sed 's/tags->//'
on the YAML files, but it's possible that it might throw up some unexpected regressions.The text was updated successfully, but these errors were encountered: