{"action":"created","issue":{"url":"https://api.github.com/repos/lesfurets/git-octopus/issues/13","repository_url":"https://api.github.com/repos/lesfurets/git-octopus","labels_url":"https://api.github.com/repos/lesfurets/git-octopus/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/lesfurets/git-octopus/issues/13/comments","events_url":"https://api.github.com/repos/lesfurets/git-octopus/issues/13/events","html_url":"https://github.com/lesfurets/git-octopus/pull/13","id":164726994,"number":13,"title":"Added chunk mode","user":{"login":"benba","id":1714536,"avatar_url":"https://avatars.githubusercontent.com/u/1714536?v=3","gravatar_id":"","url":"https://api.github.com/users/benba","html_url":"https://github.com/benba","followers_url":"https://api.github.com/users/benba/followers","following_url":"https://api.github.com/users/benba/following{/other_user}","gists_url":"https://api.github.com/users/benba/gists{/gist_id}","starred_url":"https://api.github.com/users/benba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benba/subscriptions","organizations_url":"https://api.github.com/users/benba/orgs","repos_url":"https://api.github.com/users/benba/repos","events_url":"https://api.github.com/users/benba/events{/privacy}","received_events_url":"https://api.github.com/users/benba/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-07-10T18:10:23Z","updated_at":"2016-07-14T23:14:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/lesfurets/git-octopus/pulls/13","html_url":"https://github.com/lesfurets/git-octopus/pull/13","diff_url":"https://github.com/lesfurets/git-octopus/pull/13.diff","patch_url":"https://github.com/lesfurets/git-octopus/pull/13.patch"},"body":"The chunk mode is inspired by this answer of Linus Torvalds http://marc.info/?l=linux-kernel&m=139033182525831&w=2\r\nThe goal is to make the log cleaner to follow when lots of branches are part of the octopus merge (especially if graphical tools are used to browse the history).\r\n\r\nthe option is -s (for split) and take an integer argument as parameter (which is the number of branches per chunk).\r\nIt works combined with -n or with the signal interruption.\r\n\r\nI tried to minimize the impact on the script as much as I could.\r\nNote that I also introduced a check in case the no branches match the pattern, because the script were hanging in that case (in can separate this in another PR in needed).\r\nAlso I'm not sure to have understood properly the triggeredBranch variable.\r\nIn some cases it contains a branch name, in others a sha1. Therefore I've created another variable that always the contains the sha1 to be able to reset the triggeredBranch at the correct state.\r\n\r\nLast some screenshots of the result with a dumb repo that contains 101 branches to be merged.\r\n\r\n_Without the chunk mode:_\r\n![101 branches command line without chunks](https://cloud.githubusercontent.com/assets/1714536/16715182/b046cde2-46d9-11e6-8f4b-d0b6c0a36226.png)\r\n![101 branches gitk without chunks](https://cloud.githubusercontent.com/assets/1714536/16715184/b05f46f6-46d9-11e6-91fb-dfb11b83efae.png)\r\n_With -s10:_\r\n![101 branches command line with chunks](https://cloud.githubusercontent.com/assets/1714536/16715183/b058487e-46d9-11e6-88e2-453ba4520d3b.png)\r\n![101 branches gitk with chunks](https://cloud.githubusercontent.com/assets/1714536/16715181/b0364dbe-46d9-11e6-920b-c179fafd9b0e.png)\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/lesfurets/git-octopus/13)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/lesfurets/git-octopus/issues/comments/232819384","html_url":"https://github.com/lesfurets/git-octopus/pull/13#issuecomment-232819384","issue_url":"https://api.github.com/repos/lesfurets/git-octopus/issues/13","id":232819384,"user":{"login":"apflieger","id":3698901,"avatar_url":"https://avatars.githubusercontent.com/u/3698901?v=3","gravatar_id":"","url":"https://api.github.com/users/apflieger","html_url":"https://github.com/apflieger","followers_url":"https://api.github.com/users/apflieger/followers","following_url":"https://api.github.com/users/apflieger/following{/other_user}","gists_url":"https://api.github.com/users/apflieger/gists{/gist_id}","starred_url":"https://api.github.com/users/apflieger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apflieger/subscriptions","organizations_url":"https://api.github.com/users/apflieger/orgs","repos_url":"https://api.github.com/users/apflieger/repos","events_url":"https://api.github.com/users/apflieger/events{/privacy}","received_events_url":"https://api.github.com/users/apflieger/received_events","type":"User","site_admin":false},"created_at":"2016-07-14T23:14:37Z","updated_at":"2016-07-14T23:14:37Z","body":"That's a great work. I don't have much to say. \r\nThere's an other documentation to update. I'll do it because it's a bit more complicated. I didn't write how to generate the html/manpage docs.\r\nI would appreciate if somebody can test it at LesFurets.com on different OS. @dbaeli, who could do that ? :)"}}
{"actor":{"display_login":"apflieger"}}