Allow options to have default parameters

[thecrypticace]

I saw #21 after I ran into a need for this today.

• Default parameters will now also be determined via reflection.
• Setting defaults via ->defaults(…) will now set defaults for matching options as well.

[mnapoli]

Hi @thecrypticace, this is awesome!

From reading the code I think I noticed this:

$app->command('greet [firstname] [lastname] [--age=]', function () {
    // ...
})->defaults([
    'firstname' => 'John',
    'lastname'  => 'Doe',
    '--age' => 25,
]);

works but also this:

    'age' => 25,

Is that right?

Maybe we should allow only one of those option to work, because currently there are 3 ways to define a default option value and it's one too much. Can options and arguments have identical names? If there's no chance for a collision we could only allow the simpler version:

})->defaults([
    'age' => 25,
]);

[thecrypticace]

… and it's one too much.

Yeah. I agree.

Can options and arguments have identical names?

While it looks like Symfony allows this, I just did a bit of testing and things will get squashed on in the invoker: a later defined argument / option will override an earlier one with the same name.

I've updated the PR to change this.

[mnapoli]

Thanks! Sorry for the wait

[kfriend]

Nice work, thanks @thecrypticace!