{"action":"closed","number":33,"pull_request":{"url":"https://api.github.com/repos/nodejs/node-eps/pulls/33","id":75182150,"html_url":"https://github.com/nodejs/node-eps/pull/33","diff_url":"https://github.com/nodejs/node-eps/pull/33.diff","patch_url":"https://github.com/nodejs/node-eps/pull/33.patch","issue_url":"https://api.github.com/repos/nodejs/node-eps/issues/33","number":33,"state":"closed","locked":false,"title":"Replace section 5.1 with unambiguous JavaScript grammar.","user":{"login":"jdalton","id":4303,"avatar_url":"https://avatars.githubusercontent.com/u/4303?v=3","gravatar_id":"","url":"https://api.github.com/users/jdalton","html_url":"https://github.com/jdalton","followers_url":"https://api.github.com/users/jdalton/followers","following_url":"https://api.github.com/users/jdalton/following{/other_user}","gists_url":"https://api.github.com/users/jdalton/gists{/gist_id}","starred_url":"https://api.github.com/users/jdalton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdalton/subscriptions","organizations_url":"https://api.github.com/users/jdalton/orgs","repos_url":"https://api.github.com/users/jdalton/repos","events_url":"https://api.github.com/users/jdalton/events{/privacy}","received_events_url":"https://api.github.com/users/jdalton/received_events","type":"User","site_admin":false},"body":"This PR replaces [section 5.1](https://github.com/nodejs/node-eps/blob/master/002-es6-modules.md#51-determining-if-source-is-an-es-module) of the [ES6 Module Interoperability](https://github.com/nodejs/node-eps/blob/master/002-es6-modules.md) proposal with the [UnambiguousJavaScriptGrammar](https://github.com/bmeck/UnambiguousJavaScriptGrammar) proposal.\r\n\r\n#### TL;DR\r\n\r\n* CJS and ES modules **just work** without new extensions, extra ceremony, or\r\n  excessive scaffolding\r\n* Performance is generally on par with existing CJS module loading\r\n* Performance is significantly improved for ES modules over transpilation workflows\r\n* Change JS grammars for Script and Module to be unambiguous\r\n* Determine grammar of `.js` files by parsing as one grammar and falling back to others\r\n\r\n>\r\n<img src=\"https://cloud.githubusercontent.com/assets/4303/16352075/a50ccc9a-3a1f-11e6-967e-26bae2d8d38b.gif\" width=50% height=50%>","created_at":"2016-06-24T22:26:54Z","updated_at":"2016-07-06T21:15:49Z","closed_at":"2016-07-06T21:15:49Z","merged_at":null,"merge_commit_sha":"211a9c76473efbf61a38d3b008bdeae445c43402","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/nodejs/node-eps/pulls/33/commits","review_comments_url":"https://api.github.com/repos/nodejs/node-eps/pulls/33/comments","review_comment_url":"https://api.github.com/repos/nodejs/node-eps/pulls/comments{/number}","comments_url":"https://api.github.com/repos/nodejs/node-eps/issues/33/comments","statuses_url":"https://api.github.com/repos/nodejs/node-eps/statuses/46b165a52d221fa03a5aa8ba5ac611d15a054361","head":{"label":"jdalton:unambiguous","ref":"unambiguous","sha":"46b165a52d221fa03a5aa8ba5ac611d15a054361","user":{"login":"jdalton","id":4303,"avatar_url":"https://avatars.githubusercontent.com/u/4303?v=3","gravatar_id":"","url":"https://api.github.com/users/jdalton","html_url":"https://github.com/jdalton","followers_url":"https://api.github.com/users/jdalton/followers","following_url":"https://api.github.com/users/jdalton/following{/other_user}","gists_url":"https://api.github.com/users/jdalton/gists{/gist_id}","starred_url":"https://api.github.com/users/jdalton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdalton/subscriptions","organizations_url":"https://api.github.com/users/jdalton/orgs","repos_url":"https://api.github.com/users/jdalton/repos","events_url":"https://api.github.com/users/jdalton/events{/privacy}","received_events_url":"https://api.github.com/users/jdalton/received_events","type":"User","site_admin":false},"repo":{"id":61823697,"name":"node-eps","full_name":"jdalton/node-eps","owner":{"login":"jdalton","id":4303,"avatar_url":"https://avatars.githubusercontent.com/u/4303?v=3","gravatar_id":"","url":"https://api.github.com/users/jdalton","html_url":"https://github.com/jdalton","followers_url":"https://api.github.com/users/jdalton/followers","following_url":"https://api.github.com/users/jdalton/following{/other_user}","gists_url":"https://api.github.com/users/jdalton/gists{/gist_id}","starred_url":"https://api.github.com/users/jdalton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdalton/subscriptions","organizations_url":"https://api.github.com/users/jdalton/orgs","repos_url":"https://api.github.com/users/jdalton/repos","events_url":"https://api.github.com/users/jdalton/events{/privacy}","received_events_url":"https://api.github.com/users/jdalton/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/jdalton/node-eps","description":"Node.js Enhancement Proposals for discussion on future API additions/changes to Node core","fork":true,"url":"https://api.github.com/repos/jdalton/node-eps","forks_url":"https://api.github.com/repos/jdalton/node-eps/forks","keys_url":"https://api.github.com/repos/jdalton/node-eps/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jdalton/node-eps/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jdalton/node-eps/teams","hooks_url":"https://api.github.com/repos/jdalton/node-eps/hooks","issue_events_url":"https://api.github.com/repos/jdalton/node-eps/issues/events{/number}","events_url":"https://api.github.com/repos/jdalton/node-eps/events","assignees_url":"https://api.github.com/repos/jdalton/node-eps/assignees{/user}","branches_url":"https://api.github.com/repos/jdalton/node-eps/branches{/branch}","tags_url":"https://api.github.com/repos/jdalton/node-eps/tags","blobs_url":"https://api.github.com/repos/jdalton/node-eps/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jdalton/node-eps/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jdalton/node-eps/git/refs{/sha}","trees_url":"https://api.github.com/repos/jdalton/node-eps/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jdalton/node-eps/statuses/{sha}","languages_url":"https://api.github.com/repos/jdalton/node-eps/languages","stargazers_url":"https://api.github.com/repos/jdalton/node-eps/stargazers","contributors_url":"https://api.github.com/repos/jdalton/node-eps/contributors","subscribers_url":"https://api.github.com/repos/jdalton/node-eps/subscribers","subscription_url":"https://api.github.com/repos/jdalton/node-eps/subscription","commits_url":"https://api.github.com/repos/jdalton/node-eps/commits{/sha}","git_commits_url":"https://api.github.com/repos/jdalton/node-eps/git/commits{/sha}","comments_url":"https://api.github.com/repos/jdalton/node-eps/comments{/number}","issue_comment_url":"https://api.github.com/repos/jdalton/node-eps/issues/comments{/number}","contents_url":"https://api.github.com/repos/jdalton/node-eps/contents/{+path}","compare_url":"https://api.github.com/repos/jdalton/node-eps/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jdalton/node-eps/merges","archive_url":"https://api.github.com/repos/jdalton/node-eps/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jdalton/node-eps/downloads","issues_url":"https://api.github.com/repos/jdalton/node-eps/issues{/number}","pulls_url":"https://api.github.com/repos/jdalton/node-eps/pulls{/number}","milestones_url":"https://api.github.com/repos/jdalton/node-eps/milestones{/number}","notifications_url":"https://api.github.com/repos/jdalton/node-eps/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jdalton/node-eps/labels{/name}","releases_url":"https://api.github.com/repos/jdalton/node-eps/releases{/id}","deployments_url":"https://api.github.com/repos/jdalton/node-eps/deployments","created_at":"2016-06-23T17:24:06Z","updated_at":"2016-06-24T23:14:37Z","pushed_at":"2016-07-04T16:05:57Z","git_url":"git://github.com/jdalton/node-eps.git","ssh_url":"git@github.com:jdalton/node-eps.git","clone_url":"https://github.com/jdalton/node-eps.git","svn_url":"https://github.com/jdalton/node-eps","homepage":null,"size":28,"stargazers_count":0,"watchers_count":0,"language":null,"has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"nodejs:master","ref":"master","sha":"5dae5a537c2d56fbaf23aaf2ae9da15e74474021","user":{"login":"nodejs","id":9950313,"avatar_url":"https://avatars.githubusercontent.com/u/9950313?v=3","gravatar_id":"","url":"https://api.github.com/users/nodejs","html_url":"https://github.com/nodejs","followers_url":"https://api.github.com/users/nodejs/followers","following_url":"https://api.github.com/users/nodejs/following{/other_user}","gists_url":"https://api.github.com/users/nodejs/gists{/gist_id}","starred_url":"https://api.github.com/users/nodejs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nodejs/subscriptions","organizations_url":"https://api.github.com/users/nodejs/orgs","repos_url":"https://api.github.com/users/nodejs/repos","events_url":"https://api.github.com/users/nodejs/events{/privacy}","received_events_url":"https://api.github.com/users/nodejs/received_events","type":"Organization","site_admin":false},"repo":{"id":48204104,"name":"node-eps","full_name":"nodejs/node-eps","owner":{"login":"nodejs","id":9950313,"avatar_url":"https://avatars.githubusercontent.com/u/9950313?v=3","gravatar_id":"","url":"https://api.github.com/users/nodejs","html_url":"https://github.com/nodejs","followers_url":"https://api.github.com/users/nodejs/followers","following_url":"https://api.github.com/users/nodejs/following{/other_user}","gists_url":"https://api.github.com/users/nodejs/gists{/gist_id}","starred_url":"https://api.github.com/users/nodejs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nodejs/subscriptions","organizations_url":"https://api.github.com/users/nodejs/orgs","repos_url":"https://api.github.com/users/nodejs/repos","events_url":"https://api.github.com/users/nodejs/events{/privacy}","received_events_url":"https://api.github.com/users/nodejs/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/nodejs/node-eps","description":"Node.js Enhancement Proposals for discussion on future API additions/changes to Node core","fork":false,"url":"https://api.github.com/repos/nodejs/node-eps","forks_url":"https://api.github.com/repos/nodejs/node-eps/forks","keys_url":"https://api.github.com/repos/nodejs/node-eps/keys{/key_id}","collaborators_url":"https://api.github.com/repos/nodejs/node-eps/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/nodejs/node-eps/teams","hooks_url":"https://api.github.com/repos/nodejs/node-eps/hooks","issue_events_url":"https://api.github.com/repos/nodejs/node-eps/issues/events{/number}","events_url":"https://api.github.com/repos/nodejs/node-eps/events","assignees_url":"https://api.github.com/repos/nodejs/node-eps/assignees{/user}","branches_url":"https://api.github.com/repos/nodejs/node-eps/branches{/branch}","tags_url":"https://api.github.com/repos/nodejs/node-eps/tags","blobs_url":"https://api.github.com/repos/nodejs/node-eps/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/nodejs/node-eps/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/nodejs/node-eps/git/refs{/sha}","trees_url":"https://api.github.com/repos/nodejs/node-eps/git/trees{/sha}","statuses_url":"https://api.github.com/repos/nodejs/node-eps/statuses/{sha}","languages_url":"https://api.github.com/repos/nodejs/node-eps/languages","stargazers_url":"https://api.github.com/repos/nodejs/node-eps/stargazers","contributors_url":"https://api.github.com/repos/nodejs/node-eps/contributors","subscribers_url":"https://api.github.com/repos/nodejs/node-eps/subscribers","subscription_url":"https://api.github.com/repos/nodejs/node-eps/subscription","commits_url":"https://api.github.com/repos/nodejs/node-eps/commits{/sha}","git_commits_url":"https://api.github.com/repos/nodejs/node-eps/git/commits{/sha}","comments_url":"https://api.github.com/repos/nodejs/node-eps/comments{/number}","issue_comment_url":"https://api.github.com/repos/nodejs/node-eps/issues/comments{/number}","contents_url":"https://api.github.com/repos/nodejs/node-eps/contents/{+path}","compare_url":"https://api.github.com/repos/nodejs/node-eps/compare/{base}...{head}","merges_url":"https://api.github.com/repos/nodejs/node-eps/merges","archive_url":"https://api.github.com/repos/nodejs/node-eps/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/nodejs/node-eps/downloads","issues_url":"https://api.github.com/repos/nodejs/node-eps/issues{/number}","pulls_url":"https://api.github.com/repos/nodejs/node-eps/pulls{/number}","milestones_url":"https://api.github.com/repos/nodejs/node-eps/milestones{/number}","notifications_url":"https://api.github.com/repos/nodejs/node-eps/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/nodejs/node-eps/labels{/name}","releases_url":"https://api.github.com/repos/nodejs/node-eps/releases{/id}","deployments_url":"https://api.github.com/repos/nodejs/node-eps/deployments","created_at":"2015-12-17T23:39:07Z","updated_at":"2016-07-04T18:05:45Z","pushed_at":"2016-07-06T21:15:14Z","git_url":"git://github.com/nodejs/node-eps.git","ssh_url":"git@github.com:nodejs/node-eps.git","clone_url":"https://github.com/nodejs/node-eps.git","svn_url":"https://github.com/nodejs/node-eps","homepage":null,"size":93,"stargazers_count":133,"watchers_count":133,"language":null,"has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":22,"mirror_url":null,"open_issues_count":15,"forks":22,"open_issues":15,"watchers":133,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/nodejs/node-eps/pulls/33"},"html":{"href":"https://github.com/nodejs/node-eps/pull/33"},"issue":{"href":"https://api.github.com/repos/nodejs/node-eps/issues/33"},"comments":{"href":"https://api.github.com/repos/nodejs/node-eps/issues/33/comments"},"review_comments":{"href":"https://api.github.com/repos/nodejs/node-eps/pulls/33/comments"},"review_comment":{"href":"https://api.github.com/repos/nodejs/node-eps/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/nodejs/node-eps/pulls/33/commits"},"statuses":{"href":"https://api.github.com/repos/nodejs/node-eps/statuses/46b165a52d221fa03a5aa8ba5ac611d15a054361"}},"merged":false,"mergeable":true,"mergeable_state":"clean","merged_by":null,"comments":45,"review_comments":5,"commits":1,"additions":78,"deletions":134,"changed_files":1}}
{"actor":{"display_login":"trevnorris"}}