Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

System.Text.Json should only be a dependency for .NET standard #496

Closed
thompson-tomo opened this issue Apr 16, 2024 · 3 comments · Fixed by #497
Closed

System.Text.Json should only be a dependency for .NET standard #496

thompson-tomo opened this issue Apr 16, 2024 · 3 comments · Fixed by #497
Assignees
Labels

Comments

@thompson-tomo
Copy link
Contributor

Describe the problem you are trying to solve.
I do not want STJ listed as a dependency for net 6

Available workarounds
Accept un-necessary dependency

Additional context
N/a

@abatishchev abatishchev changed the title System.text.Json should only be a dependency for net standard System.Text.Json should only be a dependency for .NET standard Apr 16, 2024
@abatishchev abatishchev self-assigned this Apr 16, 2024
@abatishchev
Copy link
Member

Hi!
Can you describe the issue more? What's wrong with a dependency to a standard library such as STJ?
Originally and till today, the library depends on NS.J. It was a community's request to add STJ. So now we have 2.
Neither is a part of the .NET Standard, which is dead anyway, the recommendation by the .NET team to target the full framework directly, this is what the library does nowadays.

@abatishchev
Copy link
Member

I'd you feel super strong about this, please consider implementing #76.

@thompson-tomo
Copy link
Contributor Author

@abatishchev we don't need to go too those lengths, simply adjusting the conditions on the package is sufficent as proposed in #497

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

2 participants