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
Add requires.tables to <layer>.yaml #1236
Add requires.tables to <layer>.yaml #1236
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it would be cleaner to use the yaml list rather than json list - makes it easier to read.
Hi @Falke-Design, I agree that tables should be in the YAML list, the same as you did for the test in OMT-T Could you please update it? Thank you |
@TomPohys should the |
Results evaluating commit 73fc779 (merged with base 5b2e434 as 7ae64ee). See run details. PostgreSQL DB size in MB: 2853 ⇒ 2851 (-0.1% change)
expand for details...
|
Thank you, it looks better now. A single item as a YAML list looks OK, and it has consistency through all files. If the |
@TomPohys done |
Thank you! |
NOTE this can only be merged after the next tools version is released.
Added required Postgres tables to the
<layer>.yaml
definition.Close: #1220
PR of tools: openmaptiles/openmaptiles-tools#370