Skip to content
New issue

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

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

Already on GitHub? # to your account

add JSON formatter and editor from VS Code #5

Merged
merged 1 commit into from
Jan 10, 2018

Conversation

sqs
Copy link
Contributor

@sqs sqs commented Dec 23, 2017

Adds the JSON formatter and editor from VS Code in src/vs/base/common/{jsonFormatter,jsonEdit}.ts, respectively, and their respective tests.

Those files have not changed in the VS Code repository since October 2016 and March 2017, respectively, which I interpret to mean they are pretty stable and safe to extract.

Feel free to close if you don't want this. 😄

Adds the JSON formatter and editor from VS Code in src/vs/base/common/{jsonFormatter,jsonEdit}.ts, respectively, and their respective tests.
@aeschli
Copy link
Contributor

aeschli commented Jan 10, 2018

That's fine with me. I guess we should also add some API to the main class. Any preference?

@aeschli aeschli merged commit 8b3ab94 into microsoft:master Jan 10, 2018
@aeschli aeschli added this to the January 2018 milestone Jan 10, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants