Skip to content

Conversation

guillaumerose
Copy link
Contributor

@guillaumerose guillaumerose commented Jan 21, 2022

This PR adds the first unit test. I am totally new to cmake. Let me know how I can improve that!

I think next step would be to extract enough function to be able to generate a full mbtiles given a osm.pbf and use it in one test.

@kleunen
Copy link
Contributor

kleunen commented Jan 22, 2022

You now have a separate cmake file, but it is better to include the cmake from the main cmake file.

Or maybe, just add the target of the test in the main cmake file.
https://github.com/systemed/tilemaker/blob/master/CMakeLists.txt

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.

2 participants