{"action":"created","issue":{"url":"https://api.github.com/repos/aspnet/dnx/issues/1816","labels_url":"https://api.github.com/repos/aspnet/dnx/issues/1816/labels{/name}","comments_url":"https://api.github.com/repos/aspnet/dnx/issues/1816/comments","events_url":"https://api.github.com/repos/aspnet/dnx/issues/1816/events","html_url":"https://github.com/aspnet/dnx/issues/1816","id":74114040,"number":1816,"title":"project.json: commented lines break \"compile\" arrays","user":{"login":"sayedihashimi","id":1283154,"avatar_url":"https://avatars.githubusercontent.com/u/1283154?v=3","gravatar_id":"","url":"https://api.github.com/users/sayedihashimi","html_url":"https://github.com/sayedihashimi","followers_url":"https://api.github.com/users/sayedihashimi/followers","following_url":"https://api.github.com/users/sayedihashimi/following{/other_user}","gists_url":"https://api.github.com/users/sayedihashimi/gists{/gist_id}","starred_url":"https://api.github.com/users/sayedihashimi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sayedihashimi/subscriptions","organizations_url":"https://api.github.com/users/sayedihashimi/orgs","repos_url":"https://api.github.com/users/sayedihashimi/repos","events_url":"https://api.github.com/users/sayedihashimi/events{/privacy}","received_events_url":"https://api.github.com/users/sayedihashimi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-05-07T20:20:45Z","updated_at":"2015-05-07T21:12:59Z","closed_at":null,"body":"Moving from https://github.com/aspnet/Tooling/issues/47\r\n\r\nFrom @mgravell\r\n\r\nThis is fine elsewhere, for example:\r\n\r\n    \"commands\": {\r\n        //\"dummy\": \"nothing\",\r\n        \"test\": \"xunit.runner.dnx\"\r\n    },\r\n\r\nWhich is then validly ignored:\r\n\r\n    D:\\Dev\\protobuf-net\\Protobuf.Tests>dnx . dummy\r\n    System.InvalidOperationException: Unable to load application or execute command\r\n    'dummy'. Available commands: test.\r\n\r\nHowever! It breaks the IDE when used in `\"compile:\"`:\r\n\r\n    \"compile\": [\r\n        //\"../Examples/Diction*.cs\",\r\n        \"../Examples/Program.cs\"\r\n    ],\r\n\r\nThis causes 3 errors:\r\n\r\n- Illegal characters in path.\r\n- NuGet Package Restore failed for one or more packages. See details in the Output window.\r\n- The design time host build failed with the following error: Illegal characters in path.\r\n\r\nAnd the \"References\" node in the solution explorer breaks (yellow hazard sign)\r\n\r\nNote: this also breaks `dnx` when used at the command line, for example:\r\n\r\n    D:\\Dev\\protobuf-net\\Protobuf.Tests>dnx . test\r\n    System.ArgumentException: Illegal characters in path.\r\n       at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)\r\n       at System.IO.Path.IsPathRooted(String path)\r\n       at Microsoft.Framework.Runtime.PatternsCollectionHelper.CreateCollection(Stri\r\n    ng projectDirectory, String[] patternsStrings)"},"comment":{"url":"https://api.github.com/repos/aspnet/dnx/issues/comments/100020899","html_url":"https://github.com/aspnet/dnx/issues/1816#issuecomment-100020899","issue_url":"https://api.github.com/repos/aspnet/dnx/issues/1816","id":100020899,"user":{"login":"muratg","id":1808428,"avatar_url":"https://avatars.githubusercontent.com/u/1808428?v=3","gravatar_id":"","url":"https://api.github.com/users/muratg","html_url":"https://github.com/muratg","followers_url":"https://api.github.com/users/muratg/followers","following_url":"https://api.github.com/users/muratg/following{/other_user}","gists_url":"https://api.github.com/users/muratg/gists{/gist_id}","starred_url":"https://api.github.com/users/muratg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/muratg/subscriptions","organizations_url":"https://api.github.com/users/muratg/orgs","repos_url":"https://api.github.com/users/muratg/repos","events_url":"https://api.github.com/users/muratg/events{/privacy}","received_events_url":"https://api.github.com/users/muratg/received_events","type":"User","site_admin":false},"created_at":"2015-05-07T21:12:59Z","updated_at":"2015-05-07T21:12:59Z","body":"@mgravell I understand the value, but based on an internal discussion, we have decided that it's not worth it to break the [spec](http://www.ietf.org/rfc/rfc4627.txt). \r\n\r\nTools like __npm__, __bower__ etc already lack the support so there was already a bit of inconsistency between the JSON files within an ASP.NET 5 project. \r\n\r\nAlso, we expect 3rd party tools to interact with `project.json` files in the future, and we can't guarantee all their parsers will have support for comments.\r\n\r\nPlease see why Douglas Crockford decided to remove them from JSON [here](https://plus.google.com/+DouglasCrockfordEsq/posts/RK8qyGVaGSr). "}}