OSM-es / CatAtom2Osm

Tool to convert INSPIRE data sets from the Spanish Cadastre ATOM Services to OSM files.
BSD 2-Clause "Simplified" License
13 stars 6 forks source link

Permite indicar un fichero de configuración #103

Closed agarciamontoro closed 2 years ago

agarciamontoro commented 2 years ago

Esta PR añade dos nuevas opciones al comando catatom2osm:

No tengo claro cómo tengo que testear esto (más allá de haberme asegurado de que make test acaba sin errores), pero estaría bien hacerlo, así que cualquier ayuda es bienvenida.

Cierra #90.

javiersanp commented 2 years ago

Gracias Alejandro. Tiene todo muy buena pinta. Voy a revisarlo, te pongo algunos comentarios y fusiono.

javiersanp commented 2 years ago

Yo me encargo de los test y te voy comentando.

agarciamontoro commented 2 years ago

Genial, gracias! :)

javiersanp commented 2 years ago

Las pruebas están en c654249. Se usa la librería mock para reemplazar con simulacros, sobre todo open.

agarciamontoro commented 2 years ago

Gracias, le echo un vistazo y así lo sé para la próxima vez :)