{"action":"created","issue":{"url":"https://api.github.com/repos/lydell/frappe/issues/1","labels_url":"https://api.github.com/repos/lydell/frappe/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/lydell/frappe/issues/1/comments","events_url":"https://api.github.com/repos/lydell/frappe/issues/1/events","html_url":"https://github.com/lydell/frappe/issues/1","id":101206796,"number":1,"title":"This looks great!","user":{"login":"rattrayalex","id":704302,"avatar_url":"https://avatars.githubusercontent.com/u/704302?v=3","gravatar_id":"","url":"https://api.github.com/users/rattrayalex","html_url":"https://github.com/rattrayalex","followers_url":"https://api.github.com/users/rattrayalex/followers","following_url":"https://api.github.com/users/rattrayalex/following{/other_user}","gists_url":"https://api.github.com/users/rattrayalex/gists{/gist_id}","starred_url":"https://api.github.com/users/rattrayalex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rattrayalex/subscriptions","organizations_url":"https://api.github.com/users/rattrayalex/orgs","repos_url":"https://api.github.com/users/rattrayalex/repos","events_url":"https://api.github.com/users/rattrayalex/events{/privacy}","received_events_url":"https://api.github.com/users/rattrayalex/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-08-15T20:13:48Z","updated_at":"2015-08-19T12:25:13Z","closed_at":null,"body":"Hi @lydell , \r\n\r\nComing from the CoffeeScript Redux ES6 thread and just wanted to say I think this looks like a terrific syntax. \r\n\r\nWanted to share a few (totally personal, this-is-all-just-me) thoughts: \r\n\r\n- `|>` is the most visually jarring thing on the page... it seems like a good idea, I'm also bothered by deleting/re-adding the `()`'s, it's just the most different. \r\n- I'd be in favor of having `isnt` -> `!==`, I think it's clear to developer that it means something different than `is not`. \r\n- `a.'prop-name'` is concerning to me... allowing more ways to access properties makes grepping harder, and I don't really see the advantage. Though I generally find that style dynamic property access to be something of an antipattern, so partly I just don't want to make it any easier than it has to be.\r\n- Automatic comma insertions would be a top feature for me, at least in the case of newlines. I don't think automatic commas when things are all the same line makes the code more clear. \r\n- Perhaps use `when` to alias `case... break`? `case-fallthrough` looks a bit cumbersome, and `case` without `break` seems confusing/misleading. `when` would signal something different from ES behavior is happening. \r\n- What are some of the \"tough decisions\" that significant whitespace would entail making? (genuine question, I'm sure there are many but I don't know what they are). \r\n- FWIW, I think you made the right call with all items in the `Intentionally left out CoffeeScript features` section. \r\n- It would probably be nicer for the Frappe linter to wrap something like ESLint, rather than making the developer use two linters. \r\n- The changes to strings is clearly the biggest departure from JavaScript. I'm not sure I see the advantage in merging `'`, `\"`, and `` ` ``, though I like the proposed newline-related behavior . ES's behavior of backticks-for-templates makes it clear to the developer when a string is a template string, which is a good thing IMO. \r\n- I also like the proposed regex behavior. \r\n- The \"runtime\" recommendation for trailing whitespace is the kind of ugly hack that indicates there's a deeper problem... But maybe just remove that recommendation :-)\r\n- Overall, this looks like a great syntax! My biggest concern is that without whitespace and automatic commas, the advantage over vanilla ES6 might not be big enough to warrant lots of people to switch.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/lydell/frappe/issues/comments/132572372","html_url":"https://github.com/lydell/frappe/issues/1#issuecomment-132572372","issue_url":"https://api.github.com/repos/lydell/frappe/issues/1","id":132572372,"user":{"login":"vendethiel","id":199499,"avatar_url":"https://avatars.githubusercontent.com/u/199499?v=3","gravatar_id":"","url":"https://api.github.com/users/vendethiel","html_url":"https://github.com/vendethiel","followers_url":"https://api.github.com/users/vendethiel/followers","following_url":"https://api.github.com/users/vendethiel/following{/other_user}","gists_url":"https://api.github.com/users/vendethiel/gists{/gist_id}","starred_url":"https://api.github.com/users/vendethiel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vendethiel/subscriptions","organizations_url":"https://api.github.com/users/vendethiel/orgs","repos_url":"https://api.github.com/users/vendethiel/repos","events_url":"https://api.github.com/users/vendethiel/events{/privacy}","received_events_url":"https://api.github.com/users/vendethiel/received_events","type":"User","site_admin":false},"created_at":"2015-08-19T12:25:13Z","updated_at":"2015-08-19T12:25:13Z","body":"I still don't like that it looks like implicit calls, but at least it's not ambiguous :)."}}