{"comment":{"url":"https://api.github.com/repos/mshenfield/malachite-migrations/comments/10205564","html_url":"https://github.com/mshenfield/malachite-migrations/commit/625a0e01cc7a332aa1ff9b05789dd13f90484fca#commitcomment-10205564","id":10205564,"user":{"login":"carrollgt91","id":1665058,"avatar_url":"https://avatars.githubusercontent.com/u/1665058?v=3","gravatar_id":"","url":"https://api.github.com/users/carrollgt91","html_url":"https://github.com/carrollgt91","followers_url":"https://api.github.com/users/carrollgt91/followers","following_url":"https://api.github.com/users/carrollgt91/following{/other_user}","gists_url":"https://api.github.com/users/carrollgt91/gists{/gist_id}","starred_url":"https://api.github.com/users/carrollgt91/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/carrollgt91/subscriptions","organizations_url":"https://api.github.com/users/carrollgt91/orgs","repos_url":"https://api.github.com/users/carrollgt91/repos","events_url":"https://api.github.com/users/carrollgt91/events{/privacy}","received_events_url":"https://api.github.com/users/carrollgt91/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"625a0e01cc7a332aa1ff9b05789dd13f90484fca","created_at":"2015-03-16T00:09:29Z","updated_at":"2015-03-16T00:09:29Z","body":"Really great work here. Love the way you destructured the inputs to the keys map so that the generator code is really clean but the underlying implementation has clear parameter names.\r\n\r\nI think that since these guys are a little more complex, they might justify a few more additional tests. I'll go ahead and merge this in and add another issue to flesh those out so we don't forget."}}