-
Notifications
You must be signed in to change notification settings - Fork 100
Closed
Labels
area: editor configsRelated to outputting IDE settings filesRelated to outputting IDE settings filesstatus: accepting prsPlease, send in a PR to resolve this! ✨Please, send in a PR to resolve this! ✨type: enhancementNew feature or requestNew feature or request
Description
🚀 Feature Request
Forking from #742: editor configs can have a lot of formatting considerations in them that we won't want to override. Spacing, sorting, comments, trailing commas or lack thereof: these are all things that should be preserved when writing a new config.
Existing Behavior
We directly write the results of JSON.stringify()
ing a new computed editor config.
Change Proposal
Use something like jsondiffpatch
to write the new config.
Note: this issue will also apply to Atom's CSON configs (#728).
KingDarBoja
Metadata
Metadata
Assignees
Labels
area: editor configsRelated to outputting IDE settings filesRelated to outputting IDE settings filesstatus: accepting prsPlease, send in a PR to resolve this! ✨Please, send in a PR to resolve this! ✨type: enhancementNew feature or requestNew feature or request