mirror of
https://github.com/valitydev/thrift.git
synced 2024-11-07 10:48:51 +00:00
3bc6424c76
Client: All Patch: Konrad Grochowski This closes #371
1.6 KiB
1.6 KiB
Process used by committers to review and submit patches
-
Make sure that there is an issue for the patch(s) you are about to commit in our Jira issue tracker
-
Check out the latest version of the source code
- git clone https://git-wip-us.apache.org/repos/asf/thrift.git thrift
- Apply the patch
- curl https://issues.apache.org/jira/... |git apply --ignore-space-change
or
- curl https://github.com//thrift/commit/.patch |git apply --ignore-space-change
-
Inspect the applied patch to ensure that all Legal aspects on Submission of Contributions (Patches) are met
-
Run the necessary unit tests and cross language test cases to verify the patch
-
Commit the patch
git --config user.name "Your Name" git --config user.email "YourApacheID@apache.org" git add -A git commit
-
The commit message should be in the format:
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 add below comment to automaticaly close GitHub request. %> This closes #XX
-
Double check the patch committed and that nothing was missed then push the patch
git status git show HEAD git push origin master
-
Resolve the jira issue and set the following for the changelog
- Component the patch is for
- fixVersion to the current version on master