Cysharp / YetAnotherHttpHandler

YetAnotherHttpHandler brings the power of HTTP/2 (and gRPC) to Unity and .NET Standard.
MIT License
376 stars 33 forks source link

UnityNuGet usage removal #57

Open bdovaz opened 8 months ago

bdovaz commented 8 months ago

@mayuki Honestly I am disappointed that you have thrown away all my work that took so long to merge (https://github.com/Cysharp/YetAnotherHttpHandler/pull/54) without creating an issue first and at least checking it with me in case I could help before making this drastic change....

With this change you may have broken the integration with OpenUPM and those of us who use UnityNuGet with OpenUPM may start having problems with conflicts and duplications.

mayuki commented 8 months ago

I apologize for potentially disrupting your work. But this change only impacts the contents of the .unitypackage.

Originally, the instructions were to install YetAnotherHttpHandler.Dependencies via UnityNuGet, with no dependencies on YetAnotherHttpHandler itself. And there are currently no plans to change this. YetAnotherHttpHandler.Dependencies for UnityNuGet will remain.

We believe that if using OpenUPM or UnityNuGet, dependencies should be installed from there and should not be resolved by the Package contained in the .unitypackage. We think it is preferable to use YetAnotherHttpHandler.Dependencies to resolve them from those package registries.