Add optional context to `resolveCallableDependencies`

[yiisoft]

Add optional context to `resolveCallableDependencies`

[dynasource]

agree. It makes sense to make Yii::$app in resolveCallableDependencies customizable

[samdark]

Yes. It's the same except https://www.yiiframework.com/news/177/yii-adopts-semver-since-version-3-0-0

[SamMousa]

Any input, i'm happy to defend the proposal if anyone has an opinion :)

[samdark]

It makes sense to me. @yiisoft/core-developers what do you think?

[samdark]

Can't do so w/o breaking BC.

[SamMousa]

Why not? Adding an optional argument is allowed right?

Anyway, I'm fine with pushing this to 2.1

[samdark]

Why not? Adding an optional argument is allowed right?

Yes.

[SamMousa]

@samdark for which version should I make a PR?

Or should this be closed since you're working on a new DI project that will change everything?

[samdark]

I'm still unsure if we'll be able to use that DI library in 3.0.0 since it may change framework too much. Will get an answer in a few weeks (hopefully).

[SamMousa]

So definitely not in 2.x. Should I make a PR for 2.0.x then?

[samdark]

Enhancements aren't accepted into 2.0 so I think it should be 3.0 after we'll decide on DI.

[SamMousa]

Ah, but won't there be a 2.1? Or we going to 3 immediately?

[samdark]

Closing since implementation is too different now.