{"action":"created","issue":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12659","labels_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12659/labels{/name}","comments_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12659/comments","events_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12659/events","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/issues/12659","id":88068042,"number":12659,"title":"Core Devs: label for PRs in the merge pipeline","user":{"login":"drbig","id":1252899,"avatar_url":"https://avatars.githubusercontent.com/u/1252899?v=3","gravatar_id":"","url":"https://api.github.com/users/drbig","html_url":"https://github.com/drbig","followers_url":"https://api.github.com/users/drbig/followers","following_url":"https://api.github.com/users/drbig/following{/other_user}","gists_url":"https://api.github.com/users/drbig/gists{/gist_id}","starred_url":"https://api.github.com/users/drbig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drbig/subscriptions","organizations_url":"https://api.github.com/users/drbig/orgs","repos_url":"https://api.github.com/users/drbig/repos","events_url":"https://api.github.com/users/drbig/events{/privacy}","received_events_url":"https://api.github.com/users/drbig/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-06-13T21:24:08Z","updated_at":"2015-06-13T23:34:05Z","closed_at":null,"body":"A simple rule: if you're working on merging a given PR add a specific label to it.\r\n\r\nThe point is anyone (including my ircbot) can then easily query github to list PRs in the merge pipeline.\r\n\r\nA label is easier to search by than by assignee, it's self-explanatory and explicit, and you can remove it if the PR needs more work (or whatever if it's no longer in the merge pipeline), but still be the assignee if you e.g. want to work with the author on making it mergable.\r\n\r\nThis should also help you to filter out open PRs that are *not* in the pipeline.\r\n\r\nWhile the total effectiveness of this scheme depends on you actually adding the label, it's not that anything's going to break if you don't add it, so I don't see any downsides. \r\n\r\nAnd as to the label itself: anything you all can agree on. 'for-merge' seems like a reasonable candidate.\r\n\r\n*(the point is still for anyone to easily filter out PRs that are in the merge pipeline; I'd be happy with any mechanism that can do that; labels just seem the easiest way to me)*\r\n\r\n<bountysource-plugin>\r\n---\r\nWant to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/21892623-core-devs-label-for-prs-in-the-merge-pipeline?utm_campaign=plugin&utm_content=tracker%2F146201&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F146201&utm_medium=issues&utm_source=github).\r\n</bountysource-plugin>"},"comment":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/comments/111760359","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/issues/12659#issuecomment-111760359","issue_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12659","id":111760359,"user":{"login":"Coolthulhu","id":1878860,"avatar_url":"https://avatars.githubusercontent.com/u/1878860?v=3","gravatar_id":"","url":"https://api.github.com/users/Coolthulhu","html_url":"https://github.com/Coolthulhu","followers_url":"https://api.github.com/users/Coolthulhu/followers","following_url":"https://api.github.com/users/Coolthulhu/following{/other_user}","gists_url":"https://api.github.com/users/Coolthulhu/gists{/gist_id}","starred_url":"https://api.github.com/users/Coolthulhu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Coolthulhu/subscriptions","organizations_url":"https://api.github.com/users/Coolthulhu/orgs","repos_url":"https://api.github.com/users/Coolthulhu/repos","events_url":"https://api.github.com/users/Coolthulhu/events{/privacy}","received_events_url":"https://api.github.com/users/Coolthulhu/received_events","type":"User","site_admin":false},"created_at":"2015-06-13T23:34:05Z","updated_at":"2015-06-13T23:34:05Z","body":"Assigning/deassigning allows quickly discerning whether someone is working on the code right now. This helps to prevent the situations when two people try to merge conflicting changes."}}