search

autofac / Autofac.ServiceFabric

Autofac integration for Azure Service Fabric. Provides service factory implementations for Actors, Stateful Services and Stateless Services.
MIT License
26 stars 27 forks source link

Split nuget package into individual packages for Stateless, Stateful and Actors #29

Closed fcastellscha closed 2 years ago

fcastellscha commented 6 years ago

Currently, this package adds dependencies to the 3 types of SF services even if only one dependency is required.

tillig commented 2 years ago

PR #60 is open for this at the moment, but I don't know if... honestly, if this really matters. It's not uncommon for framework/integration libraries to potentially add some dependencies you're not going to use. It makes it easier for the owners of the library (us) to maintain it if it's not two different packages, and if everything is versioned/updated at the same time in the integration library, there's not any actual harm. It's not entirely unlike things like ASP.NET MVC adding several references to things you may not want; it's still pay-for-play, and if you don't use it, you don't use it.

If there is some actual technological detriment to having these here ("X doesn't work if Y is present," "X performs worse if Y is also referenced-but-unused") I'd be happy to reopen this provided those explanations.