-
-
Notifications
You must be signed in to change notification settings - Fork 155
Extra name tags #1190
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
base: main
Are you sure you want to change the base?
Extra name tags #1190
Conversation
|
Full logs: https://github.com/onthegomap/planetiler/actions/runs/13499475093 |
|
I just merged #1187 - did you mean to add any other changes in this PR? |
|
Apologies. I think I got a new PR rather than appending onto the old one. #1190 contains the work to move the extra name tags logic from OmtTranslationUtils to org.onthegomap.planetiler.util.Translations. |
|
Looking at the timing, maybe you caught my changes in my repo before I made the new commit request. I didn't see the email I got from Github while I was sleeping... |



Logic for copying extra name tags from OSM to output file is now in
org.onthegomap.planetiler.util.Translations.addTranslations. I've removed the logger and logging code as suggested.There are no required changed to openmaptiles-planetiler, and the corresponding PR for that repo is now closed.