[BUG] ZiggyCreatures.FusionCache.MicrosoftHybridCache Dependency on Microsoft.Extensions.Caching.Hybrid

[pwelter34]

Describe the bug

The nuget package ZiggyCreatures.FusionCache.MicrosoftHybridCache has a dependency on Microsoft.Extensions.Caching.Hybrid. However, the HybridCache abstraction is in the Microsoft.Extensions.Caching.Abstractions package. Do you really need to reference Microsoft.Extensions.Caching.Hybrid? If you use the Microsoft.Extensions.Caching.Abstractions package, you have the advantage of using a released and supported dependency.

Expected behavior

Reference Microsoft.Extensions.Caching.Abstractions instead of Microsoft.Extensions.Caching.Hybrid

[pwelter34]

To further the discussion, do you really need the separate ZiggyCreatures.FusionCache.MicrosoftHybridCache package? ZiggyCreatures.FusionCache already has a reference to Microsoft.Extensions.Caching.Abstractions

[jodydonetti]

Hi @pwelter34

Describe the bug

The nuget package ZiggyCreatures.FusionCache.MicrosoftHybridCache has a dependency on Microsoft.Extensions.Caching.Hybrid. However, the HybridCache abstraction is in the Microsoft.Extensions.Caching.Abstractions package. Do you really need to reference Microsoft.Extensions.Caching.Hybrid? If you use the Microsoft.Extensions.Caching.Abstractions package, you have the advantage of using a released and supported dependency.

Thanks for pointing this out, I think you're right: I just need to update the current package reference to Microsoft.Extensions.Caching.Memory to 9.0.0 (which in turn depends on Microsoft.Extensions.Caching.Abstractions), so that should be enough.
Will update the package refs in the next preview.

To further the discussion, do you really need the separate ZiggyCreatures.FusionCache.MicrosoftHybridCache package? ZiggyCreatures.FusionCache already has a reference to Microsoft.Extensions.Caching.Abstractions

This is something I'm still playing with: I started separating it, now I'm finalizing all the other details, but I may end up not requiring that.
In that case (anyway before v2 goes GA, of course) I'll just mark the new separate package as deprecated and include everything into the main one.
I think I have to doublecheck with Marc (Gravell) too, because at first we agreed upon having a separate package for the integration, so maybe I'm missing something right now.

Will update, thanks again!