{"action":"created","issue":{"url":"https://api.github.com/repos/docker/docker/issues/14445","labels_url":"https://api.github.com/repos/docker/docker/issues/14445/labels{/name}","comments_url":"https://api.github.com/repos/docker/docker/issues/14445/comments","events_url":"https://api.github.com/repos/docker/docker/issues/14445/events","html_url":"https://github.com/docker/docker/pull/14445","id":93614425,"number":14445,"title":"Document and adjust reviewing process and labels","user":{"login":"icecrime","id":1564054,"avatar_url":"https://avatars.githubusercontent.com/u/1564054?v=3","gravatar_id":"","url":"https://api.github.com/users/icecrime","html_url":"https://github.com/icecrime","followers_url":"https://api.github.com/users/icecrime/followers","following_url":"https://api.github.com/users/icecrime/following{/other_user}","gists_url":"https://api.github.com/users/icecrime/gists{/gist_id}","starred_url":"https://api.github.com/users/icecrime/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icecrime/subscriptions","organizations_url":"https://api.github.com/users/icecrime/orgs","repos_url":"https://api.github.com/users/icecrime/repos","events_url":"https://api.github.com/users/icecrime/events{/privacy}","received_events_url":"https://api.github.com/users/icecrime/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/docker/docker/labels/status/3-needs-docs-review","name":"status/3-needs-docs-review","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":32,"created_at":"2015-07-07T19:54:40Z","updated_at":"2015-07-15T19:37:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/docker/docker/pulls/14445","html_url":"https://github.com/docker/docker/pull/14445","diff_url":"https://github.com/docker/docker/pull/14445.diff","patch_url":"https://github.com/docker/docker/pull/14445.patch"},"body":"This is both a documentation of the existing reviewing process (which is already mostly documented in the [MAINTAINERS](https://github.com/docker/docker/blob/master/MAINTAINERS) file but in a less readable manner), and a proposal to simplify and change our labeling process.\r\n\r\nIn particular, it:\r\n- Removes the `status/` prefix for PR workflow\r\n- Documents the escalation process\r\n- Introduces the \"specialty groups\"\r\n- Adds labels to merged PR to ease classification (for example to allow us to automate changelog generation, or a weekly \"what's new\" on the repository)\r\n\r\nThe goal is to have something we can reuse across repositories (not only those that exist today in the docker universe, but also the ones that will be created tomorrow as we cut into the Engine).\r\n\r\nI'll update the labels accordingly when/if this gets merged."},"comment":{"url":"https://api.github.com/repos/docker/docker/issues/comments/121722957","html_url":"https://github.com/docker/docker/pull/14445#issuecomment-121722957","issue_url":"https://api.github.com/repos/docker/docker/issues/14445","id":121722957,"user":{"login":"thaJeztah","id":1804568,"avatar_url":"https://avatars.githubusercontent.com/u/1804568?v=3","gravatar_id":"","url":"https://api.github.com/users/thaJeztah","html_url":"https://github.com/thaJeztah","followers_url":"https://api.github.com/users/thaJeztah/followers","following_url":"https://api.github.com/users/thaJeztah/following{/other_user}","gists_url":"https://api.github.com/users/thaJeztah/gists{/gist_id}","starred_url":"https://api.github.com/users/thaJeztah/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thaJeztah/subscriptions","organizations_url":"https://api.github.com/users/thaJeztah/orgs","repos_url":"https://api.github.com/users/thaJeztah/repos","events_url":"https://api.github.com/users/thaJeztah/events{/privacy}","received_events_url":"https://api.github.com/users/thaJeztah/received_events","type":"User","site_admin":false},"created_at":"2015-07-15T19:37:37Z","updated_at":"2015-07-15T19:37:37Z","body":"> This is both a documentation of the existing reviewing process (which is already mostly documented in the MAINTAINERS file but in a less readable manner), and a proposal to simplify and change our labeling process.\r\n\r\nShould we remove this information from the MAINTAINERS file and link to this document? I'm  worried that the information will get out of sync.\r\n\r\nIf there's a reason to keep it in both documents (e.g., because the MAINTAINERS file defines the \"rules\"), we should add a mention in this document to explain that the MAINTAINERS is \"leading\""}}