diff options
author | James E. King III <jking@apache.org> | 2019-01-30 10:29:08 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-30 10:29:08 -0500 |
commit | 06ecaae4f3ff10a607d85e2c8ccb8fcaed3e68ab (patch) | |
tree | c7bf25240c9ace3c194fd9f7cdc852980cb54fcc /CONTRIBUTING.md | |
parent | 91a4fd7076a703975f1b901fbddbeee854ae0d7b (diff) | |
download | thrift-06ecaae4f3ff10a607d85e2c8ccb8fcaed3e68ab.tar.gz |
Updated the Contributing Guide [ci skip]
Added details about trivial changes not needing a Jira ticket.
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r-- | CONTRIBUTING.md | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 7a1d710ae..7a199f73c 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -13,17 +13,17 @@ it activates the continuous integration (CI) build systems at Appveyor and Travi on a variety of Linux and Windows configurations and run all the test suites. Follow these requirements for a successful pull request: - 1. All code changes require an [Apache Jira THRIFT Issue](http://issues.apache.org/jira/browse/THRIFT) ticket. + 1. All significant changes require an [Apache Jira THRIFT Issue](http://issues.apache.org/jira/browse/THRIFT) ticket. Trivial changes such as fixing a typo or a compiler warning do not. 1. All pull requests should contain a single commit per issue, or we will ask you to squash it. - 1. The pull request title must begin with the Jira THRIFT ticket identifier, for example: + 1. The pull request title must begin with the Jira THRIFT ticket identifier if it has an associated ticket, for example: THRIFT-9999: an example pull request title 1. Commit messages must follow this pattern for code changes (deviations will not be merged): THRIFT-9999: [summary of fix, one line if possible] - Client: [language(s) affected, comma separated, use lib/ directory names please] + Client: [language(s) affected, comma separated, for example: "cpp,erl,perl"] Instructions: @@ -64,11 +64,11 @@ For unix systems, see our detailed instructions on the [Docker README](/build/do ## Contributing via Patch ## -Some changes do not require a build, for example in documentation. For changes that are not code or build related, you can submit a patch on Jira for review. To create a patch from changes in your local directory: +To create a patch from changes in your local directory: git diff > ../THRIFT-NNNN.patch -then wait for contributors or committers to review your changes, and then for a committer to apply your patch. +then wait for contributors or committers to review your changes, and then for a committer to apply your patch. This is not the preferred way to submit changes and incurs additional overhead for committers who must then create a pull request for you. ## GitHub recipes for Pull Requests ## |