{"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":11,"created_at":"2015-05-07T20:20:45Z","updated_at":"2015-05-08T22:51:29Z","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/100389400","html_url":"https://github.com/aspnet/dnx/issues/1816#issuecomment-100389400","issue_url":"https://api.github.com/repos/aspnet/dnx/issues/1816","id":100389400,"user":{"login":"dsplaisted","id":145043,"avatar_url":"https://avatars.githubusercontent.com/u/145043?v=3","gravatar_id":"","url":"https://api.github.com/users/dsplaisted","html_url":"https://github.com/dsplaisted","followers_url":"https://api.github.com/users/dsplaisted/followers","following_url":"https://api.github.com/users/dsplaisted/following{/other_user}","gists_url":"https://api.github.com/users/dsplaisted/gists{/gist_id}","starred_url":"https://api.github.com/users/dsplaisted/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dsplaisted/subscriptions","organizations_url":"https://api.github.com/users/dsplaisted/orgs","repos_url":"https://api.github.com/users/dsplaisted/repos","events_url":"https://api.github.com/users/dsplaisted/events{/privacy}","received_events_url":"https://api.github.com/users/dsplaisted/received_events","type":"User","site_admin":false},"created_at":"2015-05-08T22:51:29Z","updated_at":"2015-05-08T22:51:29Z","body":"Is there a chance of revisiting the decision not to support comments, or opening it up for some community discussion?\r\n\r\nA simple, human-readable and editable project file format is one of the big features of DNX.  It becomes a lot less readable if you can't add comments to explain what's going on, and a lot less editable if you can't quickly disable and re-enable portions of the project.json to figure out what works.\r\n\r\nI don't think people want to use comments to add parsing directives to project.json, so Douglas Crockford's reasoning doesn't seem to apply here.  Indeed, if you want comments in your source, he suggests including them there but stripping them out before the json is processed."}}