Closed alexperovich closed 7 years ago
Thanks for the PR. I am going to stick with InstancePerLifetimeScope
for this integration. The tagged instances have caused confusion in past integrations, especially when a service was to be available in multiple contexts, such as inside Service Fabric and some other application. This is also consistent with how lifetime scopes now work with ASP.NET 5 (no specific lifetime scope tag).
I have pushed 1.0.0-beta2 with your previous PR to NuGet.
This change implements an
InstancePerService
lifetime that enables one instance per service fabric service instance.