search

microsoft / semantic-kernel

Integrate cutting-edge LLM technology quickly and easily into your apps
https://aka.ms/semantic-kernel
MIT License
22.14k stars 3.3k forks source link

.Net: Leverage framework provided System.Text.Json on net 8 #5908

Open thompson-tomo opened 7 months ago

thompson-tomo commented 7 months ago

name: Feature request about: Make STJ an optional dependency based upon TFM

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.Text.Json as it is provided by the framework when also net 8

Describe alternatives you've considered Accept the additional dependency

Additional context n/a

Krzysztof318 commented 7 months ago

this is in progress already https://github.com/microsoft/semantic-kernel/pull/4421

thompson-tomo commented 7 months ago

@Krzysztof318 i have just looked through the PR and i can't identify any places where the dependencies have been optimised based on the TFM

thompson-tomo commented 6 months ago

This has not been implemented based on what was merged via #4421 hence have submitted PR as #6479