search

sogis / gretl

Contains custom gradle tasks to use in gradle builds. The custom tasks extend gradle for use as a sql-centric (geo)data etl. gretl = gradle etl
MIT License
4 stars 3 forks source link

[Ili2pgValidate] expose dataset parameter #155

Closed webrian closed 8 months ago

edigonzales commented 8 months ago

@webrian Könntest du noch einen Test machen? Ist vielleicht bissle tricky zum verstehen. Es braucht a) den Testcode: https://github.com/sogis/gretl/blob/master/gretl/src/integrationTest/java/ch/so/agi/gretl/jobs/Ili2pgValidateTest.java und b) den eigentlichen Job, z.B. https://github.com/sogis/gretl/tree/master/gretl/src/integrationTest/jobs/Ili2pgValidate

edigonzales commented 8 months ago

@webrian Ah und wenn du es im jetzigen GRETL haben willst, müsste man es noch backporten in den 2.2-Branch.

webrian commented 8 months ago

@webrian Ah und wenn du es im jetzigen GRETL haben willst, müsste man es noch backporten in den 2.2-Branch.

Ja, werde ich machen, sobald dieser PR in den Master "squashed and merged" (als ein einzelner Commit) ist. Ist das best practice?

edigonzales commented 8 months ago

@webrian Ich würde Andi fragen.

webrian commented 8 months ago

@edigonzales ist es nun ok, wenn ich merge?

edigonzales commented 8 months ago

@webrian Ja.