All,
We had several discussion about commit messages on multiple channels.
Let's discuss this here on the dev list.
Question 1: Documentation
merge HowToCommit + HowToContribute into /CONTRIBUTE.md
and include it easily with our brand new Apache CMS on the web site?
Question 2: Developer's Certificate of Origin
Today we have this commit style:
--- http://thrift.apache.org/docs/committers/HowToCommit ---
THRIFT-###:<Jira description>
Client: <component>
Patch: <Name of person contributing the patch>
Description of what was fixed or addressed.
<%
if this is a github pull request then copy the below block
from the GitHub email that came to dev@ list, this will
automatically close the GitHub pull request
%>
Github Pull Request: This closes #XX
--- http://thrift.apache.org/docs/committers/HowToCommit ---
Hmm, I did something wrong on the Lua commit...sorry!
What do you think about git best practice known from other projects to
get *Developer's Certificate of Origin*, such as here:
- https://www.kernel.org/doc/Documentation/SubmittingPatches
- http://www.denx.de/wiki/U-Boot/Patches
e.g. *git commit -s* automatically adds a Signed-off-by line to the
commit message.
Signed-off-by: Patch Creator <[email protected]>
beside of the Signed-off-by tag the following are well known:
Tested-by: Tester <[email protected]>
Reviewed-by: Reviewer <[email protected]>
Suggested-by: Idea Creator <[email protected]>
Acked-by: Component Maintainer <[email protected]>
Question 3: git am
Would it make sense to start using this?
Question 4: patch creation
What about that workflow:
git clone https://git-wip-us.apache.org/repos/asf/thrift.git thrift
cd thrift
git pull origin master
# create a local feature branch
git checkout -b THRIFT-1681
# make the changes and commit to your local feature branch
git commit -a
# format patch
git format-patch -1
# submit the patch or create a pull request
all the best!
roger
;-r
