{"action":"created","issue":{"url":"https://api.github.com/repos/BrowserSync/browser-sync/issues/1027","repository_url":"https://api.github.com/repos/BrowserSync/browser-sync","labels_url":"https://api.github.com/repos/BrowserSync/browser-sync/issues/1027/labels{/name}","comments_url":"https://api.github.com/repos/BrowserSync/browser-sync/issues/1027/comments","events_url":"https://api.github.com/repos/BrowserSync/browser-sync/issues/1027/events","html_url":"https://github.com/BrowserSync/browser-sync/issues/1027","id":139150851,"number":1027,"title":"enable bs-config.json comments","user":{"login":"myusrn","id":3043446,"avatar_url":"https://avatars.githubusercontent.com/u/3043446?v=3","gravatar_id":"","url":"https://api.github.com/users/myusrn","html_url":"https://github.com/myusrn","followers_url":"https://api.github.com/users/myusrn/followers","following_url":"https://api.github.com/users/myusrn/following{/other_user}","gists_url":"https://api.github.com/users/myusrn/gists{/gist_id}","starred_url":"https://api.github.com/users/myusrn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/myusrn/subscriptions","organizations_url":"https://api.github.com/users/myusrn/orgs","repos_url":"https://api.github.com/users/myusrn/repos","events_url":"https://api.github.com/users/myusrn/events{/privacy}","received_events_url":"https://api.github.com/users/myusrn/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-08T01:48:41Z","updated_at":"2016-03-10T06:39:52Z","closed_at":"2016-03-10T06:39:52Z","body":"### Issue details\r\n\r\nCurrently tsconfig.json, typings.json and .vscode/launch.json + settings.json + tasks.json all support use of // and /* */ comments to store details of alternative settings and notes relating to contents of those files.\r\n\r\nI tried doing this in browsersync bs-config.json, e.g. as shown in following example, and this breaks 'npm run lite-server' startup, a package that is a wrapper around browsersync, due to browsersync bs-config.json parsing errors.\r\n```\r\n{ \r\n  \"port\": 3000,  // 3000 (default) and for specific port use \"port\": 8080 or \"https\": true, \"port\": 44300\r\n  //\"files\": [\"./src/**/*.{html,htm,css,js}\"],\r\n  \"files\": [\"./**/*.{html,htm,css,js}\"],\r\n  //\"server\": { \"baseDir\": \"./src\" }\r\n  \"server\": { \"baseDir\": \"./\" }\r\n}\r\n```\r\nIs enabling similar support in browsersync bs-config.json file something that is in the plans already and if not does it sound like something that would be relevant to add?\r\n\r\n### Steps to reproduce/test case\r\n\r\nadd // and/or /* */ comments to bs-config.json file and then retry use of browsersync, e.g. by running lite-server package provided web server, and look for failure due to bs-config.json parsing.\r\n\r\n### Please specify which version of Browsersync, node and npm you're running\r\n\r\n- Browsersync [ 2.1.1  ]\r\n- Node        [  5.7.0  ]\r\n- Npm         [ 3.6.0   ]\r\n\r\n### Affected platforms\r\n\r\n- [ ] linux\r\n- [X] windows\r\n- [ ] OS X\r\n- [ ] freebsd\r\n- [ ] solaris\r\n- [ ] other _(please specify which)_\r\n\r\n### Browsersync use-case\r\n\r\n- [X] API\r\n- [ ] Gulp\r\n- [ ] Grunt\r\n- [X] CLI\r\n\r\n### If CLI, please paste the entire command below\r\n\r\nnpm run lite [ where this is configured to execute npm lite-server package that is built on top of browsersync ]\r\n\r\n### for all other use-cases, (gulp, grunt etc), please show us exactly how you're using Browsersync\r\n\r\n{Browsersync init code here}\r\n"},"comment":{"url":"https://api.github.com/repos/BrowserSync/browser-sync/issues/comments/194696884","html_url":"https://github.com/BrowserSync/browser-sync/issues/1027#issuecomment-194696884","issue_url":"https://api.github.com/repos/BrowserSync/browser-sync/issues/1027","id":194696884,"user":{"login":"shakyShane","id":1643522,"avatar_url":"https://avatars.githubusercontent.com/u/1643522?v=3","gravatar_id":"","url":"https://api.github.com/users/shakyShane","html_url":"https://github.com/shakyShane","followers_url":"https://api.github.com/users/shakyShane/followers","following_url":"https://api.github.com/users/shakyShane/following{/other_user}","gists_url":"https://api.github.com/users/shakyShane/gists{/gist_id}","starred_url":"https://api.github.com/users/shakyShane/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shakyShane/subscriptions","organizations_url":"https://api.github.com/users/shakyShane/orgs","repos_url":"https://api.github.com/users/shakyShane/repos","events_url":"https://api.github.com/users/shakyShane/events{/privacy}","received_events_url":"https://api.github.com/users/shakyShane/received_events","type":"User","site_admin":false},"created_at":"2016-03-10T06:39:52Z","updated_at":"2016-03-10T06:39:52Z","body":"Rename it to .js and use module.exports - now you have all the power of onode when generating config, along with comments :)"}}