OmniSharp / OmniSharp/csharp-language-server-protocol
Drop Newtonsoft.Json and use new System.Text.Json
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 638
- Forks
- 109
- Avg merge
- 1m
- Merged PRs (30d)
- 2
Description
With core 3 landing soon we're going to be a place to possibly start using System.Text.Json instead of Newtonsoft.Json.
Thoughts? Our serialization needs are pretty naive, so this should be a simple drop in replacement.
cc @mholo65 @tintoy @NTaylorMullen
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files, tests, or entry points are identified. Start by locating the Newtonsoft.Json dependency and all serialization call sites, then determine whether the project's protocol payloads are supported by System.Text.Json. Done means the dependency is removed, serialization behavior remains compatible, and the project's existing verification passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- api, devtools
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 32/100