ignore comments from json file while parsing json file at runtime

[anandparanjape]

JSON.NET supports comments now and K runtime needs to support that.
TSF#931826

User scenario is to add comments in project.json file. comment packages from project.json file. JSON Editor supports this.

[sayedihashimi]

Are we on track to get this in for Alpha?

[sayedihashimi]

I don't have access to edit labels, do we have a bug label that should be applied here?

[davidfowl]

This isn't being looked at for alpha.

[Eilon]

Yeah I don't think the runtime supports comments in project.json yet and it won't be for alpha. We do support JSON comments in config files (e.g. config.json) but I think that's it.

[sayedihashimi]

Thanks @Eilon, how about the next release? Any concerns with enabling it?

[Eilon]

I think it's something we'll want to enable - can you log a bug in KRuntime?

[anandparanjape]

Eilon, I opened this bug for same ask in KRuntime.

[davidfowl]

We already support /**/. Closing this as won't fix.

[anandparanjape]

Editor supports // type of comments as well. Runtime needs to support that

Sent from my Windows Phone

---

From: David Fowlermailto:[email protected]
Sent: ‎5/‎27/‎2014 2:22 PM
To: aspnet/KRuntimemailto:[email protected]
Cc: Anand Paranjapemailto:[email protected]
Subject: Re: [KRuntime] ignore comments from json file while parsing json file at runtime (#120)

We already support /**/. Closing this as won't fix.

---

Reply to this email directly or view it on GitHub:
#120 (comment)

[davidfowl]

Ok, I'll leave this in the backlog until we pick a version of JSON.NET that supports this.

[JamesNK]

Json.NET 6 supports // comments.

[anandparanjape]

Is this getting in CTP4?

[davidfowl]

It should already be in

[anandparanjape]

Yes.. it is... I just verified. Thanks David!!

[glennc]

Done, assigned to milestone.