{"action":"created","issue":{"url":"https://api.github.com/repos/angular/angular/issues/817","labels_url":"https://api.github.com/repos/angular/angular/issues/817/labels{/name}","comments_url":"https://api.github.com/repos/angular/angular/issues/817/comments","events_url":"https://api.github.com/repos/angular/angular/issues/817/events","html_url":"https://github.com/angular/angular/issues/817","id":59111868,"number":817,"title":"relying on ASI (auto semicolon insertion) in AtScript makes tooling harder","user":{"login":"alexeagle","id":47395,"avatar_url":"https://avatars.githubusercontent.com/u/47395?v=3","gravatar_id":"","url":"https://api.github.com/users/alexeagle","html_url":"https://github.com/alexeagle","followers_url":"https://api.github.com/users/alexeagle/followers","following_url":"https://api.github.com/users/alexeagle/following{/other_user}","gists_url":"https://api.github.com/users/alexeagle/gists{/gist_id}","starred_url":"https://api.github.com/users/alexeagle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexeagle/subscriptions","organizations_url":"https://api.github.com/users/alexeagle/orgs","repos_url":"https://api.github.com/users/alexeagle/repos","events_url":"https://api.github.com/users/alexeagle/events{/privacy}","received_events_url":"https://api.github.com/users/alexeagle/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/angular/angular/milestones/25","html_url":"https://github.com/angular/angular/milestones/M9:%20TypeScript","labels_url":"https://api.github.com/repos/angular/angular/milestones/25/labels","id":1086380,"number":25,"title":"M9: TypeScript","description":"","creator":{"login":"mhevery","id":111951,"avatar_url":"https://avatars.githubusercontent.com/u/111951?v=3","gravatar_id":"","url":"https://api.github.com/users/mhevery","html_url":"https://github.com/mhevery","followers_url":"https://api.github.com/users/mhevery/followers","following_url":"https://api.github.com/users/mhevery/following{/other_user}","gists_url":"https://api.github.com/users/mhevery/gists{/gist_id}","starred_url":"https://api.github.com/users/mhevery/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhevery/subscriptions","organizations_url":"https://api.github.com/users/mhevery/orgs","repos_url":"https://api.github.com/users/mhevery/repos","events_url":"https://api.github.com/users/mhevery/events{/privacy}","received_events_url":"https://api.github.com/users/mhevery/received_events","type":"User","site_admin":false},"open_issues":7,"closed_issues":1,"state":"open","created_at":"2015-04-27T22:45:18Z","updated_at":"2015-04-28T20:29:58Z","due_on":null,"closed_at":null},"comments":5,"created_at":"2015-02-26T17:59:26Z","updated_at":"2015-04-28T20:29:58Z","closed_at":"2015-04-28T20:29:58Z","body":"Stock Webstorm gives a warning on statements which don't end with semi. We could turn the warning off in the IDE, but I'd rather not rely on ASI.\r\n\r\nWe'd like to format the codebase with clang-format, but it doesn't understand javascript syntax enough to understand a statement which terminates with a linebreak instead of a semicolon. Instead it removes the linebreak, breaking the code.\r\n\r\nIn general, it's a pity to carry forward ASI into AtScript since it complicates all the tooling. Does anyone have a strong desire to omit semicolons in AtScript code?\r\n\r\n@mprobst suggests this as well. We are working on a mechanism for finding all the ASI abuses in the codebase."},"comment":{"url":"https://api.github.com/repos/angular/angular/issues/comments/97198935","html_url":"https://github.com/angular/angular/issues/817#issuecomment-97198935","issue_url":"https://api.github.com/repos/angular/angular/issues/817","id":97198935,"user":{"login":"alexeagle","id":47395,"avatar_url":"https://avatars.githubusercontent.com/u/47395?v=3","gravatar_id":"","url":"https://api.github.com/users/alexeagle","html_url":"https://github.com/alexeagle","followers_url":"https://api.github.com/users/alexeagle/followers","following_url":"https://api.github.com/users/alexeagle/following{/other_user}","gists_url":"https://api.github.com/users/alexeagle/gists{/gist_id}","starred_url":"https://api.github.com/users/alexeagle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexeagle/subscriptions","organizations_url":"https://api.github.com/users/alexeagle/orgs","repos_url":"https://api.github.com/users/alexeagle/repos","events_url":"https://api.github.com/users/alexeagle/events{/privacy}","received_events_url":"https://api.github.com/users/alexeagle/received_events","type":"User","site_admin":false},"created_at":"2015-04-28T20:29:58Z","updated_at":"2015-04-28T20:29:58Z","body":"We will use clang-format for all *.ts files in angular codebase, which means ASI won't be allowed (all semis required)\r\nOther than that, I don't think we are suggesting any restrictions, either for us or for Angular users."}}