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

Use System.Text.Json for parsing JSON instead of YamlSharp #841

Closed
darrelmiller opened this issue Apr 26, 2022 · 0 comments
Closed

Use System.Text.Json for parsing JSON instead of YamlSharp #841

darrelmiller opened this issue Apr 26, 2022 · 0 comments
Assignees
Labels
type:enhancement Enhancement request targeting an existing experience
Milestone

Comments

@darrelmiller
Copy link
Member

System.Text.Json is significantly faster than YamlSharp for parsing JSON and it doesn't have an issue with the key length limit that
YamlSharp has. #786

This will require detecting if the description being read is JSON or YAML which we can do from either the file extension or the content-type header.

We need to investigate what will be required to implement a new set of parse nodes that wrap a JsonDocument.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement Enhancement request targeting an existing experience
Projects
None yet
Development

No branches or pull requests

2 participants