{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tfjs-layers/issues/154","repository_url":"https://api.github.com/repos/tensorflow/tfjs-layers","labels_url":"https://api.github.com/repos/tensorflow/tfjs-layers/issues/154/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tfjs-layers/issues/154/comments","events_url":"https://api.github.com/repos/tensorflow/tfjs-layers/issues/154/events","html_url":"https://github.com/tensorflow/tfjs-layers/pull/154","id":317762701,"number":154,"title":"Moving serialization infrastructure to -core","user":{"login":"ericdnielsen","id":17646533,"avatar_url":"https://avatars1.githubusercontent.com/u/17646533?v=4","gravatar_id":"","url":"https://api.github.com/users/ericdnielsen","html_url":"https://github.com/ericdnielsen","followers_url":"https://api.github.com/users/ericdnielsen/followers","following_url":"https://api.github.com/users/ericdnielsen/following{/other_user}","gists_url":"https://api.github.com/users/ericdnielsen/gists{/gist_id}","starred_url":"https://api.github.com/users/ericdnielsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ericdnielsen/subscriptions","organizations_url":"https://api.github.com/users/ericdnielsen/orgs","repos_url":"https://api.github.com/users/ericdnielsen/repos","events_url":"https://api.github.com/users/ericdnielsen/events{/privacy}","received_events_url":"https://api.github.com/users/ericdnielsen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-04-25T19:28:41Z","updated_at":"2018-05-03T15:49:48Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/tensorflow/tfjs-layers/pulls/154","html_url":"https://github.com/tensorflow/tfjs-layers/pull/154","diff_url":"https://github.com/tensorflow/tfjs-layers/pull/154.diff","patch_url":"https://github.com/tensorflow/tfjs-layers/pull/154.patch"},"body":"This is part of a paired PR (one in core, one in layers).\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/tensorflow/tfjs-layers/154)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/tensorflow/tfjs-layers/issues/comments/386342702","html_url":"https://github.com/tensorflow/tfjs-layers/pull/154#issuecomment-386342702","issue_url":"https://api.github.com/repos/tensorflow/tfjs-layers/issues/154","id":386342702,"user":{"login":"caisq","id":16824702,"avatar_url":"https://avatars2.githubusercontent.com/u/16824702?v=4","gravatar_id":"","url":"https://api.github.com/users/caisq","html_url":"https://github.com/caisq","followers_url":"https://api.github.com/users/caisq/followers","following_url":"https://api.github.com/users/caisq/following{/other_user}","gists_url":"https://api.github.com/users/caisq/gists{/gist_id}","starred_url":"https://api.github.com/users/caisq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/caisq/subscriptions","organizations_url":"https://api.github.com/users/caisq/orgs","repos_url":"https://api.github.com/users/caisq/repos","events_url":"https://api.github.com/users/caisq/events{/privacy}","received_events_url":"https://api.github.com/users/caisq/received_events","type":"User","site_admin":false},"created_at":"2018-05-03T15:49:48Z","updated_at":"2018-05-03T15:49:48Z","author_association":"COLLABORATOR","body":"\n\n\n\nReview status: 0 of 29 files reviewed at latest revision, all discussions resolved, some commit checks failed.\n\n---\n\n*[package.json, line 9 at r1](https://reviewable.io/reviews/tensorflow/tfjs-layers/154#-LBag2PU55kmTOix7XhY:-LBag2PU55kmTOix7XhZ:b-czwjuh) ([raw file](https://github.com/tensorflow/tfjs-layers/blob/f47a1d8a84ef2e40f9af6c6726c0b1ffd2d6231a/package.json#L9)):*\n> ```\n> link:.yalc/@tensorflow/tfjs-core\n> ```\n\nIs this meant to be permanent? Or are you going to remove it before merging this PR?\n\nAs long as Travis tests pass, I'm happy with it. But beware that this means we'll have to start using branches for releases, because releases shouldn't depend on the HEAD.\n\n---\n\n*[package.json, line 47 at r1](https://reviewable.io/reviews/tensorflow/tfjs-layers/154#-LBafsVk88QHpGOSpjGJ:-LBafsVk88QHpGOSpjGK:b-8jfmo6) ([raw file](https://github.com/tensorflow/tfjs-layers/blob/f47a1d8a84ef2e40f9af6c6726c0b1ffd2d6231a/package.json#L47)):*\n> ```\n> \"dependencies\": {}\n> ```\n\nWhy is this necessary?\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io/reviews/tensorflow/tfjs-layers/154)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"caisq"}}