Antoine Bluchet
Antoine Bluchet
@GromNaN I'm not sure it's a good idea to have the target in the callbacks especially that if you mutate the target your results will probably be unexpected ^^. I...
I reverted my changes to the previous version with the added `$target`, in the end it's a good addition that brings flexibility. Not sure where I should put the `TargetClass`...
@joelwurtz I think it's too confusing to have `to`/`from` and `source`/`target`. I don't share the concerns as we can easily take this check as a "static" check that we can...
I'm not sure we need to anticipate these use cases quite yet as we're still a long way to go before adding performance optimizations. I suggest that we leave this...
IMHO its outside of the current scope of this PR we can always open a new issue to discuss this (or add this to the RFC issue)
I'm confused indeed, I may have added contracts instead of specific sub components I'll check that.
Except TypeInfo we didn't change much I still need to investigate.
actually all the removed components are just the replacements for their subtree split therefore we install just as much as previously. I'll keep this like that, if for some reason...