{"comment":{"url":"https://api.github.com/repos/brownplt/code.pyret.org/comments/17568729","html_url":"https://github.com/brownplt/code.pyret.org/commit/6f735349eea9f031e04759930707c45a1e318036#commitcomment-17568729","id":17568729,"user":{"login":"jpolitz","id":119891,"avatar_url":"https://avatars.githubusercontent.com/u/119891?v=3","gravatar_id":"","url":"https://api.github.com/users/jpolitz","html_url":"https://github.com/jpolitz","followers_url":"https://api.github.com/users/jpolitz/followers","following_url":"https://api.github.com/users/jpolitz/following{/other_user}","gists_url":"https://api.github.com/users/jpolitz/gists{/gist_id}","starred_url":"https://api.github.com/users/jpolitz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpolitz/subscriptions","organizations_url":"https://api.github.com/users/jpolitz/orgs","repos_url":"https://api.github.com/users/jpolitz/repos","events_url":"https://api.github.com/users/jpolitz/events{/privacy}","received_events_url":"https://api.github.com/users/jpolitz/received_events","type":"User","site_admin":false},"position":1,"line":0,"path":"src/web/js/google-apis/api-wrapper.js","commit_id":"6f735349eea9f031e04759930707c45a1e318036","created_at":"2016-05-21T16:19:42Z","updated_at":"2016-05-21T16:19:42Z","body":"Nearly all the code is already AMD.  I'll try to write up some thoughts on this overall re: new module shapes, but there are a few things going on.  Check out recent commits on `new-horizons` to see what's going on.\r\n\r\n1.  Pyret-using modules should not be AMD-shaped, because they may need to participate in a Pyret dependency chain rather than an AMD one\r\n2.  I suspect we don't want a single JS bundle for CPO, because it's useful to split up some loading as pre-Pyret and some loading as post-Pyret.\r\n3.  The \"optimization\" is really just minification, right?  That we can do (@awstlaur already did a bunch of it, but we're waiting until things settle to do it)."}}
{"actor":{"display_login":"jpolitz"}}