{"action":"created","issue":{"url":"https://api.github.com/repos/docker/docker/issues/22372","repository_url":"https://api.github.com/repos/docker/docker","labels_url":"https://api.github.com/repos/docker/docker/issues/22372/labels{/name}","comments_url":"https://api.github.com/repos/docker/docker/issues/22372/comments","events_url":"https://api.github.com/repos/docker/docker/issues/22372/events","html_url":"https://github.com/docker/docker/pull/22372","id":151462402,"number":22372,"title":"Reorganize client and cli packages","user":{"login":"dnephin","id":442180,"avatar_url":"https://avatars.githubusercontent.com/u/442180?v=3","gravatar_id":"","url":"https://api.github.com/users/dnephin","html_url":"https://github.com/dnephin","followers_url":"https://api.github.com/users/dnephin/followers","following_url":"https://api.github.com/users/dnephin/following{/other_user}","gists_url":"https://api.github.com/users/dnephin/gists{/gist_id}","starred_url":"https://api.github.com/users/dnephin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dnephin/subscriptions","organizations_url":"https://api.github.com/users/dnephin/orgs","repos_url":"https://api.github.com/users/dnephin/repos","events_url":"https://api.github.com/users/dnephin/events{/privacy}","received_events_url":"https://api.github.com/users/dnephin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/docker/docker/labels/status/2-code-review","name":"status/2-code-review","color":"fad8c7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-04-27T18:48:33Z","updated_at":"2016-04-30T00:31:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/docker/docker/pulls/22372","html_url":"https://github.com/docker/docker/pull/22372","diff_url":"https://github.com/docker/docker/pull/22372.diff","patch_url":"https://github.com/docker/docker/pull/22372.patch"},"body":"Following the split of the binaries into two package the names of files and packages was left a little scattered.  This PR moves a few things around so that the package names and file names better match their contents.\r\n\r\n33c9eda - cleanup the `cli` package by moving all the flag related structs into the `cli/flags` package\r\n\r\n30e3620 - move the `ConfigFile` struct into a new package. This will allow us to create a new default no-arg constructor for a `ConfigFile` object. Previously this was not possible because of a circular reference between `cliconfig/credentials` and `cliconfig/`\r\n\r\n2f9dd13 - consolidate the contents of `cmd/docker/client.go` into `cmd/docker/docker.go`. Now that everything in the `cmd/docker` directory is for the client there's no real value in splitting them into separate files.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/docker/docker/issues/comments/215914692","html_url":"https://github.com/docker/docker/pull/22372#issuecomment-215914692","issue_url":"https://api.github.com/repos/docker/docker/issues/22372","id":215914692,"user":{"login":"calavera","id":1050,"avatar_url":"https://avatars.githubusercontent.com/u/1050?v=3","gravatar_id":"","url":"https://api.github.com/users/calavera","html_url":"https://github.com/calavera","followers_url":"https://api.github.com/users/calavera/followers","following_url":"https://api.github.com/users/calavera/following{/other_user}","gists_url":"https://api.github.com/users/calavera/gists{/gist_id}","starred_url":"https://api.github.com/users/calavera/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/calavera/subscriptions","organizations_url":"https://api.github.com/users/calavera/orgs","repos_url":"https://api.github.com/users/calavera/repos","events_url":"https://api.github.com/users/calavera/events{/privacy}","received_events_url":"https://api.github.com/users/calavera/received_events","type":"User","site_admin":false},"created_at":"2016-04-30T00:31:36Z","updated_at":"2016-04-30T00:31:36Z","body":"LGTM"}}