Processing additional cli arguments

[ilyapuchka]

Resolves #14

[krzysztofzablocki]

What's the reason for the whitespace limitation?

[ilyapuchka]

Commander that is used to parse command line attributes. It breaks them by spaces. There is though an alternative approach. At first I just pulled all remaining positional arguments in a dictionary. But that will require them to appear only in the end of the command and without --args, like this:

sourcery <paths> arg1=value arg2 arg3=value

Also just checked that with " there could be white spaces, so I need to add trimming

[krzysztofzablocki]

Ok thanks, I'll take a look today at tweaking this a little to use the same parsing code that we use for annotation line parsing so we have exactly same behaviour in both

[krzysztofzablocki]

moved to #82

[AliSoftware]

I think @djbe added the ability to specify a flag multiple times in Stencil via a PR?
So you could do --args x 1 --args y 2?

[djbe]

No it was in Commander, and it was for repeating parameters like:
--import Something --import SomethingElse
Which resulted in a import array variable with as values ["Something", "SomethingElse"].

I think a better command line syntax would be:
--arg x=1 --arg y=2
Where you just parse the contents of the arg array.

[AliSoftware]

Ah yeah of course I meant Commander ^^
Ok so this is definitely possible now 😉

[ilyapuchka]

Hm, didn't know about that option. @krzysztofzablocki should we use this feature instead of what we have implemented?

[djbe]

I'm planning on adding this functionality into StencilSwiftKit, so we can unify this for both codebases. Something that receives an array of strings such as:

[
  "foo.bar=test1",
  "foo.baz=test2",
  "AppModule=MyApp",
  "items=a",
  "items=b"
]

And converts it into a stencil context dictionary:

[
  "AppModule": "MyApp",
  "foo": [
    "bar": "test1",
    "baz": "test2",
  ],
  "items": [
    "a",
    "b"
  ]
]

To which you could then add whatever variables your parser provides.

[krzysztofzablocki]

I'm not entirely sure this isnt overkill