kubeflow / pipelines / 1713

0%

push

travis-ci

DSL refactor (#619)

* add comments

* relocate functions in compiler to aggregate similar functions; move _build_conventional_artifact as a nested function

* reduce sanitize functions into one in the dsl.

* more comments

* move all sanitization(op name, param name) from dsl to compiler

* sanitize pipelineparam name and op_name; remove format check in pipelineparam

* remove unit test for pipelineparam op_name format checking

* fix bug: correctly replace input in the argument list

* fix bug: replace arguments with found ones

* Sanitize the file_output keys, Matches the param in the args/cmds with the whole serialized param str, Verify both param name and container name

* loosen the containerop and param name restrictions

1249 of 1406 branches covered (88.83%)

Branch coverage included in aggregate %.

2272 of 2370 relevant lines covered (95.86%)

55.73 hits per line