refactor: remove redundant $toClass from MicroMapper, allowing improvement of generic templating

[bendavies]

Follows #4

This PR removes the $toClass paramters from MicroMapper::load(), allowing for better generics templating. no stable release yet so I hope this is acceptable!

Here is a full example with no \assert(...) or @var required on the params of MicroMapper::load() and MicroMapper::populate():
https://phpstan.org/r/bae2f18f-8eb5-4377-827a-0814b86bc383
thanks to @ondrejmirtes for the help!

[stof]

As is, the analysis of the project won't be valid.
The only reason it passes is because the CI runs phpstan at level 4, which disables many rules. Try analysing this code at level 6 at least (or even at level 8 if possible). This will show you that it will likely require more changes (in the MapperConfig class for instance) to properly support those generics.

[stof]

should this be array<string, mixed> ? I think the context is expected to be an associative array.

[bendavies]

👍

[bocharsky-bw]

@bendavies are you going to continue working on this PR? I updated this branch with latest changes from main branch and fix merge complicts here

[bendavies]

@bocharsky-bw thanks for the nudge.
is there anything to do apart from fixing @stof 's comment?

[bendavies]

ok, bumping phpstan up from level 4 to at least 6 looks important.

[bocharsky-bw]

Changes look good to me here.

But it might be so that tests pass because I temporarily skipped out the related test, see #12 (comment)

If you could take a look at that failed test and make it working - would be awesome. I'm not sure how to get that done yet

[sadikoff]

hey @bendavies is there any work to be done here?