Repository: incubator-rocketmq Updated Branches: refs/heads/master 155823ef7 -> a947fe21b
[ROCKETMQ-148] Polish the contributing guide. Project: http://git-wip-us.apache.org/repos/asf/incubator-rocketmq/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-rocketmq/commit/a947fe21 Tree: http://git-wip-us.apache.org/repos/asf/incubator-rocketmq/tree/a947fe21 Diff: http://git-wip-us.apache.org/repos/asf/incubator-rocketmq/diff/a947fe21 Branch: refs/heads/master Commit: a947fe21ba9bbdea08d20661eacabe7c88677363 Parents: 155823e Author: yukon <[email protected]> Authored: Sun Mar 19 22:57:24 2017 +0800 Committer: yukon <[email protected]> Committed: Sun Mar 19 22:57:24 2017 +0800 ---------------------------------------------------------------------- CONTRIBUTING.md | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-rocketmq/blob/a947fe21/CONTRIBUTING.md ---------------------------------------------------------------------- diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index f212787..5043192 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,16 +1,17 @@ ## How To Contribute We are always very happy to have contributions, whether for trivial cleanups or big new features. -We want to have high quality, well documented codes for each programming language, as well as the surrounding [ecosystem](https://github.com/rocketmq) of integration tools that people use with RocketMQ. +We want to have high quality, well documented codes for each programming language, as well as the surrounding [ecosystem](https://github.com/apache/incubator-rocketmq-externals) of integration tools that people use with RocketMQ. Nor is code the only way to contribute to the project. We strongly value documentation, integration with other project, and gladly accept improvements for these aspects. ## Contributing code To submit a change for inclusion, please do the following: -####If the change is non-trivial please include some unit tests that cover the new functionality. -####If you are introducing a completely new feature or API it is a good idea to start a wiki and get consensus on the basic design first. -####It is our job to follow up on patches in a timely fashion. Nag us if we aren't doing our job (sometimes we drop things). + +#### If the change is non-trivial please include some unit tests that cover the new functionality. +#### If you are introducing a completely new feature or API it is a good idea to start a wiki and get consensus on the basic design first. +#### It is our job to follow up on patches in a timely fashion. Nag us if we aren't doing our job (sometimes we drop things). ## Becoming a Committer @@ -27,5 +28,5 @@ If you want to contribute the above listing points, you must abide our some prer ###### Readability - API must have Javadoc,some very important methods also must have javadoc ###### Testability - 80% above unit test coverage about main process -###### Maintainability - Comply with our [checkstyle spec](checkstyle/checkstyle.xml),and at least 3 month update frequency +###### Maintainability - Comply with our [checkstyle spec](style/rmq_checkstyle.xml), and at least 3 month update frequency ###### Deployability - We encourage you to deploy into [maven repository](http://search.maven.org/)
