danielpalme / IocPerformance

Performance comparison of .NET IoC containers
https://danielpalme.github.io/IocPerformance
Apache License 2.0
874 stars 157 forks source link

Remove MicroResolver #135

Closed MinhThienDX closed 2 years ago

MinhThienDX commented 4 years ago

Hello, I tried to remove MicroResolver. Smoke test ran fine after that.

ipjohnson commented 4 years ago

@neuecc not sure if you saw this. Do you want micro resolver removed from the benchmark?

neuecc commented 4 years ago

what reason? If it works, I think it will be useful as a performance test baseline for those who create new IoC containers. However, it has certainly not been updated and will not be updated in the future.

MinhThienDX commented 4 years ago

However, it has certainly not been updated and will not be updated in the future. I removed it because that reason. However, danielpalme can close this PR without merging if he want to keep MicroResolver.

jzabroski commented 4 years ago

It might be helpful to add a section in the benchmarks for libraries that have previously been benchmarked and were removed either because they stopped being updated or didn't support .NET Core or didn't support Microsoft DI.