-
Notifications
You must be signed in to change notification settings - Fork 4
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
[Ili2pgValidate] expose dataset parameter #155
Conversation
@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 |
@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? |
@webrian Ich würde Andi fragen. |
@edigonzales ist es nun ok, wenn ich merge? |
@webrian Ja. |
No description provided.