Re: Reminder: Please use git am when committing patches
Looks like replies are in favor of the idea that work done by our contributors should be credit appropriately and show up in places that matter. I'll make the changes in dev book to mention git am (if formatted patch) and git --author (if not formatted patch, alternatively ask the contributor to use the 'right' way and re-upload formatted patch). Here's the jira https://issues.apache.org/jira/browse/HBASE-17552. On Fri, Jan 27, 2017 at 1:08 PM, Sean Busbey wrote: > I'm a strong +1 for getting as many of our committers as possible over > to using git am or some way of making sure the contributor shows up as > the author. I agree with the previous statements about incentivizing > participation by making sure contributors get credit in places they'll > commonly look (e.g. github and other open source contribution > aggregators). Additionally, as a PMC member it is much easier to try > to build an idea of contributions to the repo over time if I can scan > through git author and sign-off tags than if I have to try to pick out > bespoke commit message details. > > > > On Thu, Jan 26, 2017 at 8:45 PM, Apekshit Sharma > wrote: > > Yes, many people who contribute to open source like their work to show up > > in git profile. I do for one. > > Past thread on this discussion: > > http://search-hadoop.com/m/HBase/YGbb14CnYDA6GX1/threaded > > > > On Thu, Jan 26, 2017 at 6:01 PM, 张铎(Duo Zhang) > > wrote: > > > >> I think the point here is github will not count the name in the commit > >> message as a contributor for the project? > >> > >> Enis Söztutar 于2017年1月27日 周五09:41写道: > >> > >> > Yep, we have been indeed using the msg (author name) historically. In > >> cases > >> > where author info is there, the guideline is to use git am --signoff. > >> > > >> > I don't think we should require git comit --author, as long as there > is > >> > attribution in the commit msg. But we can do a --author as an option. > >> > > >> > Enis > >> > > >> > On Thu, Jan 26, 2017 at 4:29 PM, 张铎(Duo Zhang) > > >> > wrote: > >> > > >> > > See here, our committer guide says that the commit message format > >> should > >> > be > >> > > HBASE-XXX XXX (the actual author). > >> > > > >> > > http://hbase.apache.org/book.html#_commit_message_format > >> > > > >> > > I think the rule was setup when we were still on svn, but we do not > >> > change > >> > > it. > >> > > > >> > > I‘m a big +1 on always using 'git am' if possible, and set > >> > > author explicitly when using 'git commit'. One more thing is that, > do > >> not > >> > > forget to add --signoff :) > >> > > > >> > > Mind opening a issue to modify the above section in hbase book, > Appy? > >> > > > >> > > Thanks. > >> > > > >> > > 2017-01-27 4:12 GMT+08:00 Chetan Khatri < > chetan.opensou...@gmail.com>: > >> > > > >> > > > Thanks , Appy. For understanding because i am very new to open > source > >> > > > contribution. > >> > > > > >> > > > On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma < > a...@cloudera.com> > >> > > > wrote: > >> > > > > >> > > > > Hi devs, > >> > > > > > >> > > > > A recent question by new contributor (thread: Git Pull Request > to > >> > > HBase) > >> > > > > and looking at history of commits, i think it is worth > re-iterating > >> > > that > >> > > > > committer should use *git am* while committing patches so that > >> > whoever > >> > > > did > >> > > > > the actual work gets appropriate credit in git history. > >> > > > > In case the patch uploaded is unformatted patch (without author > >> tag), > >> > > > > please use > >> > > > > *git commit --author=.* > >> > > > > I don't think we should be lax about it as a community if the > >> > original > >> > > > > author doesn't get appropriate credit. > >> > > > > > >> > > > > Thanks > >> > > > > -- Appy > >> > > > > > >> > > > > >> > > > >> > > >> > > > > > > > > -- > > > > -- Appy > -- -- Appy
Re: Reminder: Please use git am when committing patches
I'm a strong +1 for getting as many of our committers as possible over to using git am or some way of making sure the contributor shows up as the author. I agree with the previous statements about incentivizing participation by making sure contributors get credit in places they'll commonly look (e.g. github and other open source contribution aggregators). Additionally, as a PMC member it is much easier to try to build an idea of contributions to the repo over time if I can scan through git author and sign-off tags than if I have to try to pick out bespoke commit message details. On Thu, Jan 26, 2017 at 8:45 PM, Apekshit Sharma wrote: > Yes, many people who contribute to open source like their work to show up > in git profile. I do for one. > Past thread on this discussion: > http://search-hadoop.com/m/HBase/YGbb14CnYDA6GX1/threaded > > On Thu, Jan 26, 2017 at 6:01 PM, 张铎(Duo Zhang) > wrote: > >> I think the point here is github will not count the name in the commit >> message as a contributor for the project? >> >> Enis Söztutar 于2017年1月27日 周五09:41写道: >> >> > Yep, we have been indeed using the msg (author name) historically. In >> cases >> > where author info is there, the guideline is to use git am --signoff. >> > >> > I don't think we should require git comit --author, as long as there is >> > attribution in the commit msg. But we can do a --author as an option. >> > >> > Enis >> > >> > On Thu, Jan 26, 2017 at 4:29 PM, 张铎(Duo Zhang) >> > wrote: >> > >> > > See here, our committer guide says that the commit message format >> should >> > be >> > > HBASE-XXX XXX (the actual author). >> > > >> > > http://hbase.apache.org/book.html#_commit_message_format >> > > >> > > I think the rule was setup when we were still on svn, but we do not >> > change >> > > it. >> > > >> > > I‘m a big +1 on always using 'git am' if possible, and set >> > > author explicitly when using 'git commit'. One more thing is that, do >> not >> > > forget to add --signoff :) >> > > >> > > Mind opening a issue to modify the above section in hbase book, Appy? >> > > >> > > Thanks. >> > > >> > > 2017-01-27 4:12 GMT+08:00 Chetan Khatri : >> > > >> > > > Thanks , Appy. For understanding because i am very new to open source >> > > > contribution. >> > > > >> > > > On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma >> > > > wrote: >> > > > >> > > > > Hi devs, >> > > > > >> > > > > A recent question by new contributor (thread: Git Pull Request to >> > > HBase) >> > > > > and looking at history of commits, i think it is worth re-iterating >> > > that >> > > > > committer should use *git am* while committing patches so that >> > whoever >> > > > did >> > > > > the actual work gets appropriate credit in git history. >> > > > > In case the patch uploaded is unformatted patch (without author >> tag), >> > > > > please use >> > > > > *git commit --author=.* >> > > > > I don't think we should be lax about it as a community if the >> > original >> > > > > author doesn't get appropriate credit. >> > > > > >> > > > > Thanks >> > > > > -- Appy >> > > > > >> > > > >> > > >> > >> > > > > -- > > -- Appy
Re: Reminder: Please use git am when committing patches
Yes, many people who contribute to open source like their work to show up in git profile. I do for one. Past thread on this discussion: http://search-hadoop.com/m/HBase/YGbb14CnYDA6GX1/threaded On Thu, Jan 26, 2017 at 6:01 PM, 张铎(Duo Zhang) wrote: > I think the point here is github will not count the name in the commit > message as a contributor for the project? > > Enis Söztutar 于2017年1月27日 周五09:41写道: > > > Yep, we have been indeed using the msg (author name) historically. In > cases > > where author info is there, the guideline is to use git am --signoff. > > > > I don't think we should require git comit --author, as long as there is > > attribution in the commit msg. But we can do a --author as an option. > > > > Enis > > > > On Thu, Jan 26, 2017 at 4:29 PM, 张铎(Duo Zhang) > > wrote: > > > > > See here, our committer guide says that the commit message format > should > > be > > > HBASE-XXX XXX (the actual author). > > > > > > http://hbase.apache.org/book.html#_commit_message_format > > > > > > I think the rule was setup when we were still on svn, but we do not > > change > > > it. > > > > > > I‘m a big +1 on always using 'git am' if possible, and set > > > author explicitly when using 'git commit'. One more thing is that, do > not > > > forget to add --signoff :) > > > > > > Mind opening a issue to modify the above section in hbase book, Appy? > > > > > > Thanks. > > > > > > 2017-01-27 4:12 GMT+08:00 Chetan Khatri : > > > > > > > Thanks , Appy. For understanding because i am very new to open source > > > > contribution. > > > > > > > > On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma > > > > wrote: > > > > > > > > > Hi devs, > > > > > > > > > > A recent question by new contributor (thread: Git Pull Request to > > > HBase) > > > > > and looking at history of commits, i think it is worth re-iterating > > > that > > > > > committer should use *git am* while committing patches so that > > whoever > > > > did > > > > > the actual work gets appropriate credit in git history. > > > > > In case the patch uploaded is unformatted patch (without author > tag), > > > > > please use > > > > > *git commit --author=.* > > > > > I don't think we should be lax about it as a community if the > > original > > > > > author doesn't get appropriate credit. > > > > > > > > > > Thanks > > > > > -- Appy > > > > > > > > > > > > > > > -- -- Appy
Re: Reminder: Please use git am when committing patches
I think the point here is github will not count the name in the commit message as a contributor for the project? Enis Söztutar 于2017年1月27日 周五09:41写道: > Yep, we have been indeed using the msg (author name) historically. In cases > where author info is there, the guideline is to use git am --signoff. > > I don't think we should require git comit --author, as long as there is > attribution in the commit msg. But we can do a --author as an option. > > Enis > > On Thu, Jan 26, 2017 at 4:29 PM, 张铎(Duo Zhang) > wrote: > > > See here, our committer guide says that the commit message format should > be > > HBASE-XXX XXX (the actual author). > > > > http://hbase.apache.org/book.html#_commit_message_format > > > > I think the rule was setup when we were still on svn, but we do not > change > > it. > > > > I‘m a big +1 on always using 'git am' if possible, and set > > author explicitly when using 'git commit'. One more thing is that, do not > > forget to add --signoff :) > > > > Mind opening a issue to modify the above section in hbase book, Appy? > > > > Thanks. > > > > 2017-01-27 4:12 GMT+08:00 Chetan Khatri : > > > > > Thanks , Appy. For understanding because i am very new to open source > > > contribution. > > > > > > On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma > > > wrote: > > > > > > > Hi devs, > > > > > > > > A recent question by new contributor (thread: Git Pull Request to > > HBase) > > > > and looking at history of commits, i think it is worth re-iterating > > that > > > > committer should use *git am* while committing patches so that > whoever > > > did > > > > the actual work gets appropriate credit in git history. > > > > In case the patch uploaded is unformatted patch (without author tag), > > > > please use > > > > *git commit --author=.* > > > > I don't think we should be lax about it as a community if the > original > > > > author doesn't get appropriate credit. > > > > > > > > Thanks > > > > -- Appy > > > > > > > > > >
Re: Reminder: Please use git am when committing patches
Yep, we have been indeed using the msg (author name) historically. In cases where author info is there, the guideline is to use git am --signoff. I don't think we should require git comit --author, as long as there is attribution in the commit msg. But we can do a --author as an option. Enis On Thu, Jan 26, 2017 at 4:29 PM, 张铎(Duo Zhang) wrote: > See here, our committer guide says that the commit message format should be > HBASE-XXX XXX (the actual author). > > http://hbase.apache.org/book.html#_commit_message_format > > I think the rule was setup when we were still on svn, but we do not change > it. > > I‘m a big +1 on always using 'git am' if possible, and set > author explicitly when using 'git commit'. One more thing is that, do not > forget to add --signoff :) > > Mind opening a issue to modify the above section in hbase book, Appy? > > Thanks. > > 2017-01-27 4:12 GMT+08:00 Chetan Khatri : > > > Thanks , Appy. For understanding because i am very new to open source > > contribution. > > > > On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma > > wrote: > > > > > Hi devs, > > > > > > A recent question by new contributor (thread: Git Pull Request to > HBase) > > > and looking at history of commits, i think it is worth re-iterating > that > > > committer should use *git am* while committing patches so that whoever > > did > > > the actual work gets appropriate credit in git history. > > > In case the patch uploaded is unformatted patch (without author tag), > > > please use > > > *git commit --author=.* > > > I don't think we should be lax about it as a community if the original > > > author doesn't get appropriate credit. > > > > > > Thanks > > > -- Appy > > > > > >
Re: Reminder: Please use git am when committing patches
See here, our committer guide says that the commit message format should be HBASE-XXX XXX (the actual author). http://hbase.apache.org/book.html#_commit_message_format I think the rule was setup when we were still on svn, but we do not change it. I‘m a big +1 on always using 'git am' if possible, and set author explicitly when using 'git commit'. One more thing is that, do not forget to add --signoff :) Mind opening a issue to modify the above section in hbase book, Appy? Thanks. 2017-01-27 4:12 GMT+08:00 Chetan Khatri : > Thanks , Appy. For understanding because i am very new to open source > contribution. > > On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma > wrote: > > > Hi devs, > > > > A recent question by new contributor (thread: Git Pull Request to HBase) > > and looking at history of commits, i think it is worth re-iterating that > > committer should use *git am* while committing patches so that whoever > did > > the actual work gets appropriate credit in git history. > > In case the patch uploaded is unformatted patch (without author tag), > > please use > > *git commit --author=.* > > I don't think we should be lax about it as a community if the original > > author doesn't get appropriate credit. > > > > Thanks > > -- Appy > > >
Re: Reminder: Please use git am when committing patches
Thanks , Appy. For understanding because i am very new to open source contribution. On Fri, Jan 27, 2017 at 1:25 AM, Apekshit Sharma wrote: > Hi devs, > > A recent question by new contributor (thread: Git Pull Request to HBase) > and looking at history of commits, i think it is worth re-iterating that > committer should use *git am* while committing patches so that whoever did > the actual work gets appropriate credit in git history. > In case the patch uploaded is unformatted patch (without author tag), > please use > *git commit --author=.* > I don't think we should be lax about it as a community if the original > author doesn't get appropriate credit. > > Thanks > -- Appy >
Reminder: Please use git am when committing patches
Hi devs, A recent question by new contributor (thread: Git Pull Request to HBase) and looking at history of commits, i think it is worth re-iterating that committer should use *git am* while committing patches so that whoever did the actual work gets appropriate credit in git history. In case the patch uploaded is unformatted patch (without author tag), please use *git commit --author=.* I don't think we should be lax about it as a community if the original author doesn't get appropriate credit. Thanks -- Appy