feat!: migrate Newtonsoft.Json to System.Text.Json - Githubissues

Nongzhsh / Awesome.Net.WritableOptions

Awesome.Net.WritableOptions can update values into appsetting.json

MIT License

12 stars 4 forks source link

feat!: migrate Newtonsoft.Json to System.Text.Json #3

Closed nogic1008 closed 2 years ago

nogic1008 commented 3 years ago

⚠️ This PR contains BREAKING CHANGE!! ⚠️

Microsoft.Extensions.Configuration.Json is no longer using Newtonsoft.Json. But this package uses it.

This PR adds System.Text.Json direct dependency, and migrates to it.

close #1

Braking Changes

We need to add Microsoft.Extensions.Configuration.Json package to use IConfigurationBuilder.AddJsonFile().
- Test app includes it via Microsoft.Extensions.Configuration.UserSecrets.
Some JSON files cannot read because of difference with Newtonsoft.Json and System.Text.Json.
- See documents

Nongzhsh commented 2 years ago

Thx. Maybe we should create new branch to support .net core 3.*