{"action":"created","issue":{"url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/issues/72","repository_url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests","labels_url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/issues/72/labels{/name}","comments_url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/issues/72/comments","events_url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/issues/72/events","html_url":"https://github.com/gluster/glusterfs-patch-acceptance-tests/pull/72","id":189945119,"number":72,"title":"first line of commit message cannot be empty","user":{"login":"mchangir","id":12027325,"avatar_url":"https://avatars.githubusercontent.com/u/12027325?v=3","gravatar_id":"","url":"https://api.github.com/users/mchangir","html_url":"https://github.com/mchangir","followers_url":"https://api.github.com/users/mchangir/followers","following_url":"https://api.github.com/users/mchangir/following{/other_user}","gists_url":"https://api.github.com/users/mchangir/gists{/gist_id}","starred_url":"https://api.github.com/users/mchangir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mchangir/subscriptions","organizations_url":"https://api.github.com/users/mchangir/orgs","repos_url":"https://api.github.com/users/mchangir/repos","events_url":"https://api.github.com/users/mchangir/events{/privacy}","received_events_url":"https://api.github.com/users/mchangir/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-11-17T05:49:05Z","updated_at":"2016-11-17T08:18:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/pulls/72","html_url":"https://github.com/gluster/glusterfs-patch-acceptance-tests/pull/72","diff_url":"https://github.com/gluster/glusterfs-patch-acceptance-tests/pull/72.diff","patch_url":"https://github.com/gluster/glusterfs-patch-acceptance-tests/pull/72.patch"},"body":"the very first line of commit message cannot be empty\r\nthis patch enforces the constraint"},"comment":{"url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/issues/comments/261183165","html_url":"https://github.com/gluster/glusterfs-patch-acceptance-tests/pull/72#issuecomment-261183165","issue_url":"https://api.github.com/repos/gluster/glusterfs-patch-acceptance-tests/issues/72","id":261183165,"user":{"login":"mchangir","id":12027325,"avatar_url":"https://avatars.githubusercontent.com/u/12027325?v=3","gravatar_id":"","url":"https://api.github.com/users/mchangir","html_url":"https://github.com/mchangir","followers_url":"https://api.github.com/users/mchangir/followers","following_url":"https://api.github.com/users/mchangir/following{/other_user}","gists_url":"https://api.github.com/users/mchangir/gists{/gist_id}","starred_url":"https://api.github.com/users/mchangir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mchangir/subscriptions","organizations_url":"https://api.github.com/users/mchangir/orgs","repos_url":"https://api.github.com/users/mchangir/repos","events_url":"https://api.github.com/users/mchangir/events{/privacy}","received_events_url":"https://api.github.com/users/mchangir/received_events","type":"User","site_admin":false},"created_at":"2016-11-17T08:18:44Z","updated_at":"2016-11-17T08:18:44Z","body":"it doesn't hurt to be strict :)\nits just a matter of getting used to the rules\n\nmost people are following this rule anyway\nwe're just saying: it has to be so ... period\n\nthere are already examples of people ignoring what rfc.sh says or\npoints out as warnings and errors\nif rfc.sh seems an annoyance, people could manually resort to\n\n$ git pull --rebase\n$ git push origin HEAD:refs/for/$branch/bug-nnnnnnn\n\nand we'll have no control over it anwyay\n\nand what good does the flexibility to have empty line(s) above the\nactual subject line help towards productivity anyway ?\n(if that's the bottom-line we are addressing)\n\nI've even started pondering if we should run checkpatch.pl at Gerrit\nend as well just in case :)\n\nMilind\n\nOn 11/17/2016 11:32 AM, Nigel Babu wrote:\n> Is this actually possible? On my machine, I can leave the first line\n> empty, but it counts the second line as first line.\n>\n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/gluster/glusterfs-patch-acceptance-tests/pull/72#issuecomment-261163094>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/ALeFvUSxFDFmMKGz1YlpeREx-a2AbPGVks5q--3YgaJpZM4K04FM>.\n>\n"}}
{"actor":{"display_login":"mchangir"}}