{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/21991","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/21991/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/21991/comments","events_url":"https://api.github.com/repos/apache/spark/issues/21991/events","html_url":"https://github.com/apache/spark/pull/21991","id":347487471,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA2MTAwNDc2","number":21991,"title":"[SPARK-25018] [Infra] Use `Co-authored-by` and `Signed-off-by` git trailer in `merge_spark_pr.py`","user":{"login":"dbtsai","id":1134574,"node_id":"MDQ6VXNlcjExMzQ1NzQ=","avatar_url":"https://avatars1.githubusercontent.com/u/1134574?v=4","gravatar_id":"","url":"https://api.github.com/users/dbtsai","html_url":"https://github.com/dbtsai","followers_url":"https://api.github.com/users/dbtsai/followers","following_url":"https://api.github.com/users/dbtsai/following{/other_user}","gists_url":"https://api.github.com/users/dbtsai/gists{/gist_id}","starred_url":"https://api.github.com/users/dbtsai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dbtsai/subscriptions","organizations_url":"https://api.github.com/users/dbtsai/orgs","repos_url":"https://api.github.com/users/dbtsai/repos","events_url":"https://api.github.com/users/dbtsai/events{/privacy}","received_events_url":"https://api.github.com/users/dbtsai/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":16,"created_at":"2018-08-03T18:03:46Z","updated_at":"2018-08-06T22:14:40Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/21991","html_url":"https://github.com/apache/spark/pull/21991","diff_url":"https://github.com/apache/spark/pull/21991.diff","patch_url":"https://github.com/apache/spark/pull/21991.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nIn [Linux community](https://git.wiki.kernel.org/index.php/CommitMessageConventions), `Co-authored-by` and `Signed-off-by` git trailer have been used for awhile. \r\n\r\nUntil recently, Github adopted `Co-authored-by` to include the work of co-authors in the profile contributions graph and the repository's statistics. It's a convention for recognizing multiple authors, and can encourage people to collaborate in OSS communities.\r\n\r\nGit provides a command line tools to read the metadata to know who commits the code to upstream, but it's not as easy as having `Signed-off-by` as part of the message so developers can find who is the relevant committers who can help with certain part of the codebase easier.\r\n\r\nFor a single author PR, I purpose to use `Authored-by` and `Signed-off-by`, so the message will look like\r\n\r\n```\r\nAuthored-by: Author's name <author@example.com>\r\nSigned-off-by: Committer's name <committer@example.com>\r\n```\r\n\r\nFor a multi-author PR, I purpose to use `Lead-authored-by:` and `Co-authored-by:` for the lead author and co-authors. The message will look like\r\n\r\n```\r\nLead-authored-by: Lead Author's name <leadauthor@example.com>\r\nCo-authored-by: CoAuthor's name <coauthor@example.com>\r\nSigned-off-by: Committer's name <committer@example.com>\r\n```\r\n\r\nIt's also useful to include `Reviewed-by:` to give credits to the people who participate on the code reviewing. We can add this in the next iteration. "},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/410870037","html_url":"https://github.com/apache/spark/pull/21991#issuecomment-410870037","issue_url":"https://api.github.com/repos/apache/spark/issues/21991","id":410870037,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMDg3MDAzNw==","user":{"login":"AmplabJenkins","id":4031291,"node_id":"MDQ6VXNlcjQwMzEyOTE=","avatar_url":"https://avatars3.githubusercontent.com/u/4031291?v=4","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2018-08-06T22:14:40Z","updated_at":"2018-08-06T22:14:40Z","author_association":"NONE","body":"Test FAILed.\nRefer to this link for build results (access rights to CI server needed): \nhttps://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94305/\nTest FAILed."}}
{"actor":{"display_login":"AmplabJenkins"}}