spectre.console icon indicating copy to clipboard operation
spectre.console copied to clipboard

Remove System.Memory dependency from net 6+

Open thompson-tomo opened this issue 1 year ago • 3 comments

Is your feature request related to a problem? Please describe. I want to minimise dependencies in my project by utilising framework dependencies wherever possible

Describe the solution you'd like I want the package to not have an explicit dependency on System.Memory on net 6+.

Describe alternatives you've considered Accept the additional dependency

Additional context n/


Please upvote :+1: this issue if you are interested in it.

thompson-tomo avatar Apr 16 '24 23:04 thompson-tomo

@thompson-tomo Spectre.Console does not have a dependency on System.Memory on net6.0. It has a conditional dependency on that package for netstandard2.0.

CleanShot 2024-04-17 at 01 29 58@2x

patriksvensson avatar Apr 16 '24 23:04 patriksvensson

Ahh, I had not checked the preview packages and only looked at the release versions.

thompson-tomo avatar Apr 17 '24 02:04 thompson-tomo

Hmm, we might actually need to investigate this. Seems like packaging doesn't respect the target framework condition for some reason: https://www.nuget.org/packages/Spectre.Console/

CleanShot 2024-04-17 at 09 51 58@2x

patriksvensson avatar Apr 17 '24 07:04 patriksvensson

Closing this as it is now not being included.

thompson-tomo avatar May 21 '24 23:05 thompson-tomo