Make kedro viz run --pipeline consistent with kedro run --pipeline

[astrojuanlu]

I think kedro viz run --pipeline should be consistent with how kedro run --pipeline behaves. This behavior was surprising also for me.

Originally posted by @astrojuanlu in #1733 (comment)

[astrojuanlu]

More context in the linked issue. Essentially, spaces are valid for pipeline names in Kedro Viz, but not for Kedro Framework. So kedro run --pipeline="Reporting stage" would not work, but kedro viz run --pipeline="Reporting stage" does.

I liberally opened this on Kedro Viz, seemingly implying that Viz should converge with Framework. But maybe it should be the other way around? (:

[rashidakanchwala]

@astrojuanlu - do we need to do work on Kedro-Viz, or will this be a Kedro framework ticket then?

[astrojuanlu]

It's for us to decide what to do cc @merelcht

[merelcht]

We'll need to have a look at what consistency between the CLI on Framework and Viz means. Is this the only command with inconsistency? And how do other similar arguments get treated on the Framework side? Maybe changing this to be consistent with Viz would mean breaking consistency with how things are done across the CLI on Framework.

[rashidakanchwala]

We discuss this ticket when @astrojuanlu has a chance to look at this more closely.

[astrojuanlu]

Can we do with this issue the same we did with kedro-org/kedro#2259 ? Hence reframe it to "look for inconsistencies between Kedro and Kedro-Viz", prioritise it as low, and move it to the backlog.

[rashidakanchwala]

@rashidakanchwala to create an issue on highlighting inconsistencies between kedro cli and kedro-viz cli.

[rashidakanchwala]

Closing this ticket in favour of this #2165