Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

vscode: Parse settings with comments #146

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rzr
Copy link

@rzr rzr commented Oct 25, 2019

If needed developer can revert the removed comments
at the cost of loosing JSON compliance.

Bug: #104
Forwarded: https://github.com/Microsoft/generator-docker/pulls
Change-Id: I80f8b290b846f62abaf5b99d424a8cfdfd9f84a1
Signed-off-by: Philippe Coval p.coval@samsung.com

If needed developer can revert the removed comments
at the cost of loosing JSON compliance.

The alternative would be to use an other non standard JSON.parse

Bug: microsoft#104
Forwarded: microsoft#146
Change-Id: I80f8b290b846f62abaf5b99d424a8cfdfd9f84a1
Signed-off-by: Philippe Coval <p.coval@samsung.com>
@msftclas
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

❌ rzr sign now
You have signed the CLA already but the status is still pending? Let us recheck it.

@rzr rzr force-pushed the sandbox/rzr/review/master branch from eeb3359 to c4f1e58 Compare October 25, 2019 16:50
Base automatically changed from master to main February 23, 2021 22:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants